Managing ownership in VectorSchemaRoot#addVector, recent changes miss the main fault.
- Vorherrschende Sprache
- Java
- Sterne
- 94
- Forks
- 152
- Ø Merge
- 3 T. 16 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
A recent change in VectorSchemaRoot#addVector provided a simple improvement, but completely misses a major shortcoming of this routine and, almost certainly, others like it. Based on the interest in that issue (#301), this issue might be of interest to @axreldable and @jbonofre.
One cannot change the `FieldVector`s belonging to `VectorSchemaRoot`s by simply rearranging the elements of their `List`s without considering the ownership of their underlying buffers. This can cause various problems, particularly when `VectorSchemaRoot`s are closed. This is why `TransferPair` exists.
I would probably need some help with improving this as I'm quite new to using Apache Arrow (and very new to using GitHub). But I have written a possibly improved implementation if this would be of interest and would be open to collaborating on rectifying this bug.
Beitragsleitfaden
Rechercherichtung
Beginne mit VectorSchemaRoot#addVector und lies die zugehörige Diskussion in Issue #301. Untersuche, wie sich das Umordnen von FieldVectors auf den Besitz ihrer zugrunde liegenden Puffer auswirkt und wie sich VectorSchemaRoots beim Schließen verhalten. Erledigt ist die Aufgabe, wenn die Besitzübertragung korrekt behandelt wird und das Schließen von Roots nicht mehr die beschriebenen Pufferprobleme verursacht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- data
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 45/100