"Conflicting values in rows" throws general Error

Hi Joachim,

 

We do run into this error from time to time. In our case, until now it was always a developer error. Here is an example:

 

Let say you have following classes: Item, ItemStoragePlace, DocumentLineItem.

 

Primary key of Item: ITEMNO (char)

Primary key of ItemStoragePlace: ITEMNO (char) and STORAGEPLACENO (char)

Primary key of DocumentLineItem: OID (integer)

 

DocumentLineItem has a one to one relationship to Item (item):

join from: ITEMNO to: ITEM.ITEMNO

 

and ItemStoragePlace (itemStoragePlace):

join from: ITEMNO to: ITEMSTORAGEPLACE.ITEMNO

join from: STORAGEPLACENO to: ITEMSTORAGEPLACE.STORAGEPLACENO

 

Note: The same field ITEMNO is used in two relationships.

 

Lets assume we have following items with primary key ITEMNO: IT01, IT02

 

Item storage places with primary key ITEMNO, STORAGEPLACENO:

 

IT01, ST01

IT02, ST01

 

If you create a new DocumentLineItem and set item to IT01, set itemStoragePlace to IT02, ST01, you would get this error.

 

In our case, we were fortunately able to duplicate the problem in development image. So we were able to track down the issue.

 

This error in our case occurs if the same field is used in two different joins and if we provide Glorp different values for the same field.

 

Regards,

Madhu.

From: <a href="javascript:" target="_blank" gdf-obfuscated-mailto="o3PfxOqfBQAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">glorp...@... <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="o3PfxOqfBQAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">glorp...@...> On Behalf Of jtuchel
Sent: Friday, 8 May 2020 09:27
To: glorp-group <<a href="javascript:" target="_blank" gdf-obfuscated-mailto="o3PfxOqfBQAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">glorp...@...>
Subject: "Conflicting values in rows" throws general Error

 

In our production system we get "Inconsistent values in rows" from time to time when we save updated objects in a UnitOfWork. It is thrown in FieldValueWrapper>>#unifyWith:
Needless to say: I have never managed to get this error in a dev image...

 

First, we have no idea what could cause these problems. It currently seems as if these "just happen from time to time" with no obivous pattern. I have the feeling that is not true, but we've been hunting for a while now - without success. So if anybody has an idea what can cause these problems, please let me know.

 

Second, we'd like to add some walback output for these situations, so we can possibly learn more about the situation. For this it would be nice if we could catch this error.

Unfortunately, and as I have stated before, Glorp is not a great example of how to throw and catch exceptions. The failing method throws a generel Error instance instead of a GlorpError subclass.

 

I'd like to suggest replacing this with a specialized Error, most likely a new subclass of GlorpError, so that we can catch and handle just this problem.

 

• What do people think of this Class name: GlorpUpdateConflictError? What parameters would
• How can I suggest this to Glorp maintainers?

I've implemented a first idea of this in VA Smalltalk, how can I feed this back and discuss it with Glorp maintainers?

 

The implementation simply introduces a new Subclass of GlorpError and I changed FieldValueWrapper>>#unifyWith: with:

 

unifyWith: targetWrapper
    (self containsWrapper: targetWrapper) ifTrue: [^self].
    (targetWrapper containsWrapper: self) ifTrue: [^self].
    targetWrapper hasValue ifFalse: [^targetWrapper replaceWith: self].
    self hasValue ifFalse: [^self replaceWith: targetWrapper].
    self contents = targetWrapper contents
        ifFalse: [GlorpUpdateConflictError signal: ('Conflicting values in rows (location: FieldValueWrapper>>#unifyWith:) self=%1 target=%2' bindWith: self contents asString with: targetWrapper contents asString)].
    targetWrapper replaceWith: self.

 

What do people think of this? What would you store in this error object for debugging/analyisng the problem.

It enables us to catch this error individually and write a walback file with some history of the problem.

 

 

Joachim

--
You received this message because you are subscribed to the Google Groups "glorp-group" group.
To unsubscribe from this group and stop receiving emails from it, send an email to <a href="javascript:" target="_blank" gdf-obfuscated-mailto="o3PfxOqfBQAJ" rel="nofollow" onmousedown="this.href=&#39;javascript:&#39;;return true;" onclick="this.href=&#39;javascript:&#39;;return true;">glorp...@googlegroups.com.
To view this discussion on the web visit <a href="https://groups.google.com/d/msgid/glorp-group/0805be95-903b-46e3-bb5c-2659e4a36366%40googlegroups.com?utm_medium=email&amp;utm_source=footer" target="_blank" rel="nofollow" onmousedown="this.href=&#39;https://groups.google.com/d/msgid/glorp-group/0805be95-903b-46e3-bb5c-2659e4a36366%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;" onclick="this.href=&#39;https://groups.google.com/d/msgid/glorp-group/0805be95-903b-46e3-bb5c-2659e4a36366%40googlegroups.com?utm_medium\x3demail\x26utm_source\x3dfooter&#39;;return true;">https://groups.google.com/d/msgid/glorp-group/0805be95-903b-46e3-bb5c-2659e4a36366%40googlegroups.com.