Hibernate-reactive: Ausnahme beim Entfernen mehrerer Elemente aus der geordneten einbettbaren Sammlung

Erstellt am 22. Feb. 2021  ·  7Kommentare  ·  Quelle: hibernate/hibernate-reactive

Ausführen zusätzlicher Tests, um die neueste Reactive-Sammlungspersistenzlogik aus den @OrderColumn #581- Änderungen zu überprüfen. Hatte ein Problem beim Entfernen von mehr als 1 Element während eines flush().

Siehe: OrderedEmbeddableCollectionTest

Ausnahme: [Exception 0] io.vertx.core.impl.NoStackTraceThrowable: Die Anzahl der auszuführenden Parameter sollte mit der erwarteten Anzahl von Parametern = [2] übereinstimmen, aber die tatsächliche Anzahl ist [3].
[Exception 1] io.vertx.core.VertxException: Transaktion bereits abgeschlossen

Scheint, dass das Array Object[] der Parameterwerte ein Object[3] anstelle von Object[2] für das zweite gelöschte Element im neuen abstrakten Persister zurückgibt: https://github.com/hibernate/hibernate-reactive/blob/ fe9dfbaa07e4b71fe85e949a45ac23c91c26709b/hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/collection/impl/ReactiveAbstractCollectionPersister.java#L237

bug
Quelle
picture blafond
👍1

Alle 7 Kommentare

Es könnte sich lohnen, den Test mit dem Hibernate ORM-Kern zu arbeiten, nur um durchzugehen und zu sehen, was dort passiert.

gavinking picture gavinking am 22. Feb. 2021

In ORMs deleteRows ändert sich das int offset = 1; nie.

In Reactive übergeben wir den Offset an den deleteRowsParamValues( entry, index+1, id, session ) Aufruf, der auf einem inkrementierten index Wert basiert. Scheint, dass wir die Konzepte von i , index , loc und offset in ORM falsch kombinieren.

Der i scheint Aktualisierungs- und Einfügemethoden bereitzustellen, um zu wissen, welcher Sammlungseintrag bearbeitet wird, daher sollte er nicht an den Löschvorgang gebunden/verwendet werden.

blafond picture blafond am 22. Feb. 2021
👍1

Yup, es sieht so aus, als ob die Logik nur falsch portiert wurde.

gavinking picture gavinking am 22. Feb. 2021

Ich denke, nur die Änderung von index+1" in 1 spiegelt die ORM-Logik wider

blafond picture blafond am 22. Feb. 2021
👍1

Groß!

gavinking picture gavinking am 22. Feb. 2021

Guter Fang!

@blafond : Ich habe dir das zugewiesen, da du anscheinend schon eine Lösung hast

DavideD picture DavideD am 23. Feb. 2021

Danke, @blafond

gavinking picture gavinking am 23. Feb. 2021
War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

hantsy picture hantsy  ·  7Kommentare
DavideD picture DavideD  ·  31Kommentare
Thomodachi picture Thomodachi  ·  15Kommentare
Sanne picture Sanne  ·  12Kommentare
aguibert picture aguibert  ·  28Kommentare