Skip to content

SQL: Lower default JDBC frame size. - #5409

Merged
fjy merged 1 commit into
apache:masterfrom
gianm:sql-lower-avatica-frame-size
Feb 21, 2018
Merged

fjy merged 1 commit into
apache:masterfrom
gianm:sql-lower-avatica-frame-size

Conversation

@gianm

@gianm gianm commented Feb 21, 2018

Copy link
Copy Markdown
Contributor

The previous default of 100,000 was a bit excessive and could easily
lead to OOM errors on "select *" style queries.

The previous default of 100,000 was a bit excessive and could easily
lead to OOM errors on "select *" style queries.

@fmarelli fmarelli left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ok

@nishantmonu51 nishantmonu51 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, please make sure we mention this in next version release notes.

@nishantmonu51 nishantmonu51 added this to the 0.13.0 milestone Feb 21, 2018
@fjy
fjy merged commit f3796bc into apache:master Feb 21, 2018
gianm added a commit to implydata/druid-public that referenced this pull request Mar 3, 2018
The previous default of 100,000 was a bit excessive and could easily
lead to OOM errors on "select *" style queries.
gianm added a commit to implydata/druid-public that referenced this pull request Mar 12, 2018
The previous default of 100,000 was a bit excessive and could easily
lead to OOM errors on "select *" style queries.
@gianm
gianm deleted the sql-lower-avatica-frame-size branch September 23, 2022 19:28
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
The previous default of 100,000 was a bit excessive and could easily
lead to OOM errors on "select *" style queries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants