Skip to content

Fix row access after materializing query result columns - #1081

Open
fallintoplace wants to merge 1 commit into
ClickHouse:mainfrom
fallintoplace:fix/query-result-row-cache
Open

fallintoplace wants to merge 1 commit into
ClickHouse:mainfrom
fallintoplace:fix/query-result-row-cache

Conversation

@fallintoplace

Copy link
Copy Markdown

What

  • Allow result_rows after result_columns.

Why

  • Reading columns first raises StreamClosedError on later row access, even though the data is cached.

Implementation

  • Build rows from cached columns on demand.
  • Cover both access orders, empty results, multiple blocks and source cleanup.
  • Add a changelog entry.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant