pingcap/tidb

executor.Next() should not call chid.Open() or child.Close()

Open

#4.983 geöffnet am 2. Nov. 2017

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Go (6.186 Forks)batch import
help wanted

Repository-Metriken

Stars
 (40.090 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 14T 4h) (346 gemergte PRs in 30 T)

Beschreibung

calling Open() or Close() during Next() makes our executors un-maintainable, for example, if we comment out the following lines then make test, our ut failes ... https://github.com/pingcap/tidb/blob/0616a2fd1511e2206086c212261d93b1106605c7/executor/executor.go#L345

or https://github.com/pingcap/tidb/blob/0616a2fd1511e2206086c212261d93b1106605c7/executor/join.go#L88

Open is used to initialize executors and allocate resources for every executor Close is used to deallocate the resource adopted by every executor, avoid memory leak

Contributor Guide

Research-Richtung
Untersuche die Referenzen in executor/executor.go und executor/join.go, um das aktuelle Muster zu verstehen. Bestimme die korrekten Stellen, um Open und Close aufzurufen, ohne den Next Vertrag zu verletzen. Stelle sicher, dass bestehende Tests nach dem Refactoring bestehen, wie in der Issue vermerkt, dass Tests mit dem aktuellen Code fehlschlagen. Berücksichtige die breitere Executor Hierarchie, um Konsistenz zu wahren.
Tech Stack
go
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60