bug: Support non-string (bool/int) attribute values in Tango
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 65/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- go
- Ambito
- backend, distributed-systems
Direzione di ricerca
Start by examining mapper/mapper.go lines 69-77 to see the current string-only filter. Review proto/tango.proto to understand the Metadata structure and plan the additive changes. Then, look at core/itg/graph/graph.go for interning logic and internal/targetdiff/compare.go line 142 for attribute comparison. A good first step is to run existing tests to ensure they pass before modifying the type handling.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Tango only propagates string-valued Bazel attributes. mapper/mapper.go:69-77 filters ingestion to Attribute_STRING only — bool/int/other types are dropped before reaching the graph, and the proto (proto/tango.proto) has no field to carry them anyway.
Effect: internal/targetdiff/compare.go:142 can't see non-string attribute changes, so a target whose only real change is e.g. a boolean flip still gets flagged as changed (via hash) but is misclassified as INDIRECT instead of DIRECT.
Proposed Solution
- Add typed value mapping(s) to Metadata in proto/tango.proto (additive, backward compatible).
- Remove the STRING-only filter in mapper/mapper.go; route by attr.GetType().
- Update interning in core/itg/graph/graph.go to handle typed values, not just GetStringValue().
- Extend attributesChanged in internal/targetdiff/compare.go to compare typed values.
- Lingua principale
- Go
- Stelle
- 23
- Fork
- 11
- Merge medio
- 1g 10h
- PR unite (30g)
- 18
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 uber/tango
-
Support Bazel repository contents cacheForse già presa @xytan0056 l’ha presa 18 giorni fa. Apertabug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
uber/tango#338 · 3 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
race in core/bazelForse di nuovo libera @sywhang l’ha presa 136 giorni fa e non c’è nessuna pull request aperta. Aperta
uber/tango#134 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
I maintainer di solito rispondono entro 1 giorno
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
evener-transcript-v2-upgrade tool still scans transcript files through bufio's 4 KiB default bufferAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
prime-radiant-inc/evener#6370 ·
I maintainer di solito rispondono entro 1 giorno
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFApertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Mend: dependency security vulnerability
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Mend: dependency security vulnerability
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
ManageIQ/manageiq-pods#1476 ·
I maintainer di solito rispondono entro 1 giorno