refactor: unify ResultSet implementations on Arrow-backed path#175

Merged

mkaufmann merged 24 commits into

mainfrom

moritz/unify-result-sets

May 20, 2026

Commits on May 15, 2026

refactor: unify ResultSet implementations on Arrow-backed path
mkaufmann
committed
refactor: address PR review — share vector builder and use in-memory ArrowStreamReader
mkaufmann
committed
fix: close allocator on StreamingResultSet.of failure in query path
mkaufmann
committed
fix: range-check narrowing integer setters in VectorPopulator
mkaufmann
committed
refactor: namespace jdbc type-name field-metadata key
mkaufmann
committed
test: pin jdbc type-name fallback path
mkaufmann
committed
test: pin getObject(int, Map) null/empty fast path
mkaufmann
committed
fix: validate row arity in MetadataResultSets.of
mkaufmann
committed
refactor: drop empty-batch workaround in MetadataResultSets
mkaufmann
committed
test: merge DataCloudMetadataResultSetTest into MetadataResultSetsTest
mkaufmann
committed
refactor: drop StreamingResultSet getObject(Class) bandaid
mkaufmann
committed
refactor: trim docstrings and prune redundant ResultSet-shape tests
mkaufmann
committed
refactor: collapse StreamingResultSet.of factories into one
mkaufmann
committed
refactor: rename StreamingResultSet to DataCloudResultSet, drop marker interface
mkaufmann
committed
fix: declare TYPE_INFO boolean columns as BOOL and tighten VarCharVectorSetter
mkaufmann
committed
fix: chain reader and allocator close exceptions via addSuppressed
mkaufmann
committed
fix: hoist queryId before allocator construction in DataCloudStatement
mkaufmann
committed
fix: close allocator on ArrowStreamReader constructor failure and unify cap
mkaufmann
committed
fix: implement INTEGER → boolean coercion in BaseIntVectorAccessor
mkaufmann
committed
fix: make DataCloudResultSet.close idempotent across cursor.close failures
mkaufmann
committed
fix: range-check unscaled value in DecimalVectorSetter
mkaufmann
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

refactor: unify ResultSet implementations on Arrow-backed path#175

refactor: unify ResultSet implementations on Arrow-backed path#175
mkaufmann merged 24 commits into
mainfrom
moritz/unify-result-sets

Commits on May 15, 2026

refactor: unify ResultSet implementations on Arrow-backed path

refactor: address PR review — share vector builder and use in-memory ArrowStreamReader

fix: close allocator on StreamingResultSet.of failure in query path

fix: range-check narrowing integer setters in VectorPopulator

refactor: namespace jdbc type-name field-metadata key

test: pin jdbc type-name fallback path

test: pin getObject(int, Map) null/empty fast path

fix: validate row arity in MetadataResultSets.of

refactor: drop empty-batch workaround in MetadataResultSets

test: merge DataCloudMetadataResultSetTest into MetadataResultSetsTest

refactor: drop StreamingResultSet getObject(Class) bandaid

refactor: trim docstrings and prune redundant ResultSet-shape tests

refactor: collapse StreamingResultSet.of factories into one

refactor: rename StreamingResultSet to DataCloudResultSet, drop marker interface

fix: declare TYPE_INFO boolean columns as BOOL and tighten VarCharVectorSetter

fix: chain reader and allocator close exceptions via addSuppressed

fix: hoist queryId before allocator construction in DataCloudStatement

fix: close allocator on ArrowStreamReader constructor failure and unify cap

fix: implement INTEGER → boolean coercion in BaseIntVectorAccessor

fix: make DataCloudResultSet.close idempotent across cursor.close failures

fix: range-check unscaled value in DecimalVectorSetter

Commits on May 18, 2026

test: pin reader-before-allocator close order in cursor test

refactor: drop the typeName override channel

fix: tighten BaseIntVectorAccessor.getBoolean to spec-strict 0/1 only