Add a smoke test for DuckDB in Travis CI - #18
Merged
Merged
Conversation
fm4v
added a commit
to ClickHouse/sqlancer
that referenced
this pull request
Jun 24, 2026
…ate oracles (#15,17,18,22) Four self-contained, reader-safe oracles (every projected wire value toString-wrapped so the client-v2 RowBinary reader never sees a raw container/Variant/AggregateFunction column; default-on, --flag gated): - #15 MapTupleContainer Map/Tuple/Array extractions == Java ground truth (private fixture) - #17 GeoMetamorphic pointInPolygon/polygonArea/greatCircleDistance/intersection metamorphic identities, integer coords, 1e-6 float tolerance - sqlancer#18 VariantSubcolumn Variant/Dynamic/JSON subcolumn roundtrip (toString-wrapped reads only; raw-column projection emission stays gated off per plan) - sqlancer#22 AggregateStateExpansion finalizeAggregation(arrayReduce('<agg>State',groupArray(x)))==<agg>(x) for sum/min/max/uniqExact/quantileExact/groupArray + AggregatingMergeTree arm Young-type unavailability (Variant/JSON/Dynamic/geo) tolerated to IgnoreMe. Wired into factory/options/run-sqlancer.sh; mvn compile green. Probe-on-head: geo function names (polygonAreaCartesian etc.), Variant subcolumn syntax (v.Int64 vs variantElement), arrayReduce/finalizeAggregation signatures.
fm4v
added a commit
to ClickHouse/sqlancer
that referenced
this pull request
Jun 24, 2026
- sqlancer#22 AggregateStateExpansion: AggregatingMergeTree arm used finalizeAggregation(sumMerge(s)) which double-finalizes (Code 43 ILLEGAL_TYPE_OF_ARGUMENT); sumMerge(s)/uniqExactMerge(u) already return the finalized scalar. Verified on head: sumMerge=10, uniqExactMerge=3. - ClickHouseProvider: pin allow_experimental_{variant,dynamic,json}_type + allow_experimental_vector_similarity_index per connection so the VariantSubcolumn (sqlancer#18) and VectorIndexRecall (sqlancer#30) fixtures CREATE reliably (the oracle-issued SET could land on a different pooled connection than the CREATE). All four settings confirmed to exist on head. Probe (head 26.6.1.743) confirmed all other 27 new-oracle constructs work as written: Int128/Int256/Decimal(P,S), geo fns, windowFunnel/sequenceCount/retention, lagInFrame, WITH ROLLUP/GROUPING/TOTALS, LIMIT WITH TIES/BY, *EXCEPT/APPLY/COLUMNS, SEMI/ANTI/ANY, CoalescingMergeTree, Set/Join/joinGet, remote(), tokenbf_v1, vector_similarity, TTL DELETE, unhex/base64/IPv4/IPv6 roundtrips.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.