[Chore] Remove row-based query from Stream, Measure and Trace

Aperta
  • Qualcosa non va?
#13,998 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Ambito
databases

Direzione di ricerca

Inizia cercando riferimenti basati sulle righe in banyand/internal/query e nelle posizioni correlate del query-engine, quindi esamina le definizioni dell’API pubblica o di proto, i test, i fixture, i benchmark, i flag della CLI e la documentazione in docs/concept/ e docs/api-reference.md. Il lavoro è completato quando il codice, i test, i flag e la documentazione di row-mode sono stati rimossi, il comportamento di compatibilità è stato deciso e la suite di test esistente passa sul percorso columnar/vectorized.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

chore database

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

  1. Research — enumerate call sites and the public surface that exposes row-mode (CLI flags, HTTP/gRPC fields, env vars).
  2. Removal — delete the row-mode code, its tests, and benchmarks.
  3. Compatibility — if any documented option still names row-mode, return a clear error / 410 Gone for a transition period (decision in design step).
  4. Tests — make sure CI is green after deletion; update coverage reports.
  5. 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)
Lingua principale
Java
Stelle
25k
Fork
6.6k
Merge medio
12h 41m
PR unite (30g)
48

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di apache/skywalking

Tutte le issue di apache/skywalking

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.