bug: Support non-string (bool/int) attribute values in Tango
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 65/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- go
- Área
- backend, distributed-systems
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Go
- Estrellas
- 23
- Forks
- 11
- Merge medio
- 1 d 10 h
- PR fusionados (30 d)
- 18
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de uber/tango
-
Support Bazel repository contents cachePosiblemente ocupada @xytan0056 la tomó hace 18 días. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
uber/tango#338 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
race in core/bazelQuizá libre de nuevo @sywhang la tomó hace 136 días y no hay ningún pull request abierto. Abierto
Los mantenedores suelen responder en 1 día
-
implement default SHELL strategyAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
Los mantenedores suelen responder en 1 día
Todos los issues de uber/tango
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
evener-transcript-v2-upgrade tool still scans transcript files through bufio's 4 KiB default bufferAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
prime-radiant-inc/evener#6370 ·
Los mantenedores suelen responder en 1 día
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Mend: dependency security vulnerability
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Mend: dependency security vulnerability
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ManageIQ/manageiq-pods#1476 ·
Los mantenedores suelen responder en 1 día