Skip to content

[JDBC] StatementImpl - #2591

Merged
chernser merged 6 commits into
mainfrom
jdbc_statement_2532
Sep 19, 2025
Merged

[JDBC] StatementImpl #2591
chernser merged 6 commits into
mainfrom
jdbc_statement_2532

Conversation

@chernser

@chernser chernser commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Fixes maxRows problem using max_result_rows setting that doesn't guarantee hard limit and may cause exception if user is not allowed to change settings
  • More tests for maxRows added
  • Added check for unsupported method and arguments using feature manager
  • Reviewed methods that are implemented according to JDBC

Closes #2532
Closes #2582

Checklist

Delete items not relevant to your PR:

  • Closes #
  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

Loading
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.

[JDBC-V2] setMaxRows() doesn't work when result_overflow_mode is readonly [JDBC] StatementImpl Improvements

2 participants