apache / apache/hugegraph-computer

[Feature] messages.next() returns a different address object

Offen
#277 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
feature
Vorherrschende Sprache
Java
Sterne
65
Forks
52
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

> always return the same Object is reduce memory pressure and avoid GC

Yep~ But semantically, `messages.next()` returns a different address object more intuitive.
Otherwise, someone who call this function downstream need to clearly know that the same object is returned here, which is not logically intuitive and easy to make mistakes.

Do we have some more elegant solutions?

_Originally posted by @diaohancai in https://github.com/apache/incubator-hugegraph-computer/issues/275#issuecomment-1779336049_

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die Implementierung und die Aufrufer von messages.next() zu überprüfen, da das Issue keine Dateien oder Tests nennt. Vergleiche das aktuelle Verhalten bei der Objektwiederverwendung mit der angeforderten Semantik von Adressobjekten und definiere ein testbares Abschlusskriterium, bevor du etwas änderst.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
distributed-systems
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.