[FEA] Rename `smaller`/`larger` to `right`/`left` in `sort_merge_join`
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Leggi cpp/src/join/sort_merge_join.cu e cpp/src/join/sort_merge_join.hpp; l’issue identifica entrambi i file e i nomi da aggiornare. Confronta le implementazioni di join vicine o le issue collegate (#22382, #22464, #22723, #22789, #22790) per verificare la convenzione right/left. Il lavoro è completato quando i nomi interni e i commenti usano coerentemente right/left e il comportamento rimane invariato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Is your feature request related to a problem? Please describe.
The sort_merge_join internals name the two sides smaller and larger (e.g. smaller_order, unique_smaller_rows, smaller_indices, larger_numrows), and some comments say build/probe. Neither is accurate: the join object always preprocesses the right table regardless of its size, and left tables are always the probe side. The other joins already use right/left (#22382, #22464, #22723, #22789, #22790).
Describe the solution you'd like
Rename smaller/larger and build/probe to right/left in cpp/src/join/sort_merge_join.cu and cpp/src/join/sort_merge_join.hpp, including comments and parameter names. Internal APIs only, no behavior change.
New sort-merge join code should follow the same convention in the meantime.
Describe alternatives you've considered
Keep the existing names, leaving sort_merge_join inconsistent with the other joins.
- Lingua principale
- C++
- Stelle
- 9.8k
- Fork
- 1.2k
- Merge medio
- 3g 17h
- PR unite (30g)
- 306
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di NVIDIA/cudf
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 61/100
I maintainer di solito rispondono entro 2 giorni
-
feature request Python
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 2 giorni
-
[FEA] cudf-polars: `str.slice` rejects a column or expression as the offsetForse di nuovo libera @AVS0703 l’ha presa 7 giorni fa e non c’è nessuna pull request aperta. Apertacudf-polars feature request good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
NVIDIA/cudf#24982 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
[FEA] cudf-polars: support `min_horizontal` and `max_horizontal` on StringForse già presa @lance-lii l’ha presa 8 giorni fa. Apertacudf-polars feature request good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
NVIDIA/cudf#24981 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
[BUG] Replacing False with `cudf.NA` leaves the boolean value unchangedForse già presa @pcbeingused333 l’ha presa 5 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
NVIDIA/cudf#24894 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 3 giorni
-
test(tools): two testkit cache tests fail unless GOCACHE and GOMODCACHE are in the environmentApertabacklog
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
EchoTools/nevr-runtime#2385 ·
I maintainer di solito rispondono entro 1 giorno
-
type:bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
tensorflow/tflite-micro#3995 ·
I maintainer di solito rispondono entro 1 giorno
-
CPU update 1.31xApertaspeed-report
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
mw00/project-maya#961 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100