[Chore] Remove row-based query from Stream, Measure and Trace
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Área
- databases
Línea de trabajo
Empieza buscando referencias basadas en filas en banyand/internal/query y en las ubicaciones relacionadas del query-engine; después inspecciona las definiciones de la API pública o proto, las pruebas, los fixtures, los benchmarks, los flags de la CLI y la documentación en docs/concept/ y docs/api-reference.md. Se considera terminado cuando se hayan eliminado el código, las pruebas, los flags y la documentación de row-mode, se haya decidido el comportamiento de compatibilidad y la suite de pruebas existente pase en la ruta columnar/vectorized.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Remove the legacy row-based query code path from the Stream, Measure and Trace query engines in BanyanDB.
Motivation
Row-based query is the old (per-row / per-data-point) execution path. The newer columnar / vectorized execution model has been the default for some time, and the row-based path is now dead weight: duplicated tests, larger binary, and a maintenance tax. Removing it shrinks the surface area and forces callers onto the supported code path.
Scope
- Identify every reference to the row-based query path in:
banyand/internal/query(or wherever the query engines live — verify during research).- Public API / proto definitions, if row-mode is exposed there.
- Tests, fixtures, benchmarks.
- Remove the code paths, their flags / options, and any still-supported APIs.
- Update docs (
docs/concept/,docs/api-reference.md) to drop row-mode mentions. - Ensure all existing tests pass on the remaining code path.
Tasks
- Research — enumerate call sites and the public surface that exposes row-mode (CLI flags, HTTP/gRPC fields, env vars).
- Removal — delete the row-mode code, its tests, and benchmarks.
- Compatibility — if any documented option still names row-mode, return a clear error / 410 Gone for a transition period (decision in design step).
- Tests — make sure CI is green after deletion; update coverage reports.
- Docs — sweep docs and examples.
Acceptance Criteria
- No remaining row-mode code, tests, or flags in the repo.
- All previously-passing tests still pass on the columnar/vectorized path.
- Docs no longer reference row-mode.
- Binary size / build time measurably reduced.
Related
- Parent milestone: BanyanDB - 0.12.0 (#264)
- Lenguaje dominante
- Java
- Estrellas
- 25k
- Forks
- 6.6k
- Merge medio
- 12 h 56 min
- PR fusionados (30 d)
- 47
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 apache/skywalking
-
[Bug] Standalone multi-group Measure aggregation returns one row per groupPosiblemente ocupada @hanahmily la tomó hoy. Abiertobug database
apache/skywalking#14198 · 2 asignados ·
Los mantenedores suelen responder en 1 día
-
[Bug] [BanyanDB] MCP server ignores .env values and silently accepts malformed numeric/TRANSPORT settingsPosiblemente ocupada @wanjinhao1 la tomó hace 1 día. Abiertobug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
apache/skywalking#14194 ·
Los mantenedores suelen responder en 1 día
-
[Bug] BanyanDB trace block metadata format changed in 0.11 without a file format version bumpPosiblemente ocupada @hanahmily la tomó hace 10 días. Abiertobug database
apache/skywalking#14169 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
chore CLI database
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
apache/skywalking#14168 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[BanyanDB] Stream write callback reports success for events it did not storeQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
apache/skywalking#14158 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de apache/skywalking
Issues similares
-
API `/api/dataverses/{identifier}/dataset/linkingDataverses?alreadyLinking=true` returns at most 1 collectionPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoType: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
javaoperatorsdk.client.oauth-token has no effect on the Kubernetes clientPosiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
MCP add-on: JSON responses with non-ASCII characters are truncated (wrong Content-Length)Posiblemente ocupada @dhanushbs10 la tomó hoy. Abiertoadd-on bug in:mcp
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Map.Coordinate does not implement proper equalsPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertovaadin-map
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
vaadin/flow-components#10586 ·
Los mantenedores suelen responder en 1 día
-
WeReadInstaller writes the launcher database on the main thread at handover (Activity pause timeout)Abiertobug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100