This is an automated email from the ASF dual-hosted git repository.

hanahmily pushed a commit to branch vectorized-query
in repository https://gitbox.apache.org/repos/asf/skywalking-banyandb.git

commit 1cd7e768c641cf966c62a716c4d963e7ec39dbe0
Author: Hongtao Gao <[email protected]>
AuthorDate: Thu May 14 04:03:15 2026 +0000

    docs(test/integration): record GroupBy+Agg bridge bug behind vec 
aggregation flag (G8d.3)
    
    The MAX investigation showed both states fail: HEAD dispatch panics at
    BatchAggregation.fold (TypedColumn[int64] assertion on a passthrough
    source), and threading opts.GroupBy/Agg into storage returns empty
    results. The schema/storage native-column bridge from 253c5653 isn't
    end-to-end correct yet, so the parity-gate flag stays off and the
    comment captures the deeper diagnosis for whoever picks this up.
---
 test/integration/standalone/query/vectorized_test.go | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/test/integration/standalone/query/vectorized_test.go 
b/test/integration/standalone/query/vectorized_test.go
index f577b7fac..05c0b0e3c 100644
--- a/test/integration/standalone/query/vectorized_test.go
+++ b/test/integration/standalone/query/vectorized_test.go
@@ -89,11 +89,14 @@ var _ = ginkgo.Describe("vectorized parity", 
ginkgo.Ordered, func() {
                config := setup.PropertyClusterConfig(dfWriter)
                addr, _, closeFn := setup.ClosableStandalone(config, path, 
ports,
                        "--measure-vectorized-enabled=true",
-                       // G8d.3 will add 
--measure-vectorized-aggregation-enabled=true
-                       // here once the MAX-specific egress parity gap is 
closed.
-                       // 8 of 9 GroupBy+Agg cases already pass with the gate 
on; the
-                       // remaining failure (group_max timeout) is tracked 
separately
-                       // before this flag flips in the parity gate too.
+                       // G8d.3 left 
--measure-vectorized-aggregation-enabled=false here
+                       // because the egress parity work surfaced a deeper bug 
than
+                       // just field naming: GroupBy+Agg vec dispatch panics at
+                       // BatchAggregation.fold (line 268, *TypedColumn[int64] 
assertion
+                       // against a passthrough source column) regardless of 
whether
+                       // storage threads opts.GroupBy/Agg. The schema/storage 
native-
+                       // column bridge in commit 253c5653 is not yet 
end-to-end
+                       // correct. Re-flip this flag when that root cause is 
closed.
                )
                stopFn = func() {
                        closeFn()

Reply via email to