Skip to content

[CALCITE-7516] Add STREAMABLE_SCANNABLE so that CsvSchema can use CsvStreamScannableTable#4932

Open
sixcorners wants to merge 1 commit into
apache:mainfrom
sixcorners:patch-1
Open

[CALCITE-7516] Add STREAMABLE_SCANNABLE so that CsvSchema can use CsvStreamScannableTable#4932
sixcorners wants to merge 1 commit into
apache:mainfrom
sixcorners:patch-1

Conversation

@sixcorners

@sixcorners sixcorners commented May 10, 2026

Copy link
Copy Markdown

Jira Link

CALCITE-7516

Changes Proposed

This lets CsvSchema use CsvStreamScannableTables.

@sonarqubecloud

Copy link
Copy Markdown

@mihaibudiu mihaibudiu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add at least one unit test which certifies that this indeed works.

@mihaibudiu mihaibudiu added the more-tests-needed Need more tests to make the patch complete label May 11, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@calcite.apache.org list. Thank you for your contributions.

@github-actions github-actions Bot added the stale label Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

more-tests-needed Need more tests to make the patch complete stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants