Deduplicate the three field-id reassignment walkers
I maintainer di solito rispondono entro 3 giorni
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 55/100
Direzione di ricerca
Start by comparing ReassignFieldIds in crates/iceberg/src/spec/schema/id_reassigner.rs with assign_fresh_ids and assign_fresh_ids_to_type in crates/iceberg/src/transaction/update_schema.rs, then inspect spec/schema/assign_fresh_ids.rs. Run the existing schema tests, especially those covering map key/value ordering. Done means the walkers and apply_to_identifier_fields/apply_to_aliases helpers are shared without changing either reassignment order or the tested behavior.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We now have three near-identical walkers that rebuild a schema's field tree while reassigning ids:
ReassignFieldIdsassign_fresh_ids/assign_fresh_ids_to_typeinupdate_schema.rsspec/schema/assign_fresh_ids.rs, added in #3066
The apply_to_identifier_fields and apply_to_aliases helpers are identical between the first and third.
Suggestion: parameterise ReassignFieldIds with a strategy for picking the new id for an old id (default fresh, overridable to check a base schema by name first), and share the apply_to_* helpers.
Worth preserving: the map arms differ on purpose. ReassignFieldIds recurses the key subtree before assigning the value id, while assign_fresh_ids resolves both ids first to match Java. Tests pin both orders.
Raised in review of #3066.
- Lingua principale
- Rust
- Stelle
- 1.4k
- Fork
- 612
- Merge medio
- 4g 35m
- PR unite (30g)
- 71
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- 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 apache/iceberg-rust
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
apache/iceberg-rust#3551 ·
I maintainer di solito rispondono entro 3 giorni
-
REST create_table omits partition spec-id; Java-based servers (Polaris) reject itForse già presa @ninglinLiu l’ha presa 2 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
apache/iceberg-rust#3469 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
-
V1 manifest lists written by Java lose added/existing/deleted file countsForse già presa @kacper-li-airspace-intelligence l’ha presa 13 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
apache/iceberg-rust#3445 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
apache/iceberg-rust#3359 ·
I maintainer di solito rispondono entro 3 giorni
-
Row filter keeps every row for `<` and `<=` on a null column that is missing from the data fileForse già presa @mattfaltyn l’ha presa 5 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
apache/iceberg-rust#3355 · 1 reazione ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di apache/iceberg-rust
Issue simili
-
QuartoHub: have the Document name before the filename in the browser tabForse già presa @cscheid l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
quarto-dev/q2#930 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
muse P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
HerringtonDarkholme/hd-lang#923 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
longbridge/gpui-kit#3712 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation enhancement SIEM
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
ADORSYS-GIS/wazuh-agent-status#265 ·
I maintainer di solito rispondono entro 2 giorni