Skip to content

Increase default domain-compaction-threshold for phoenix, clickhouse#14029

Merged
raunaqmorarka merged 3 commits intotrinodb:masterfrom
raunaqmorarka:inc-domain
Sep 9, 2022
Merged

Increase default domain-compaction-threshold for phoenix, clickhouse#14029
raunaqmorarka merged 3 commits intotrinodb:masterfrom
raunaqmorarka:inc-domain

Conversation

@raunaqmorarka
Copy link
Copy Markdown
Member

@raunaqmorarka raunaqmorarka commented Sep 7, 2022

Description

Increase default domain-compaction-threshold for phoenix, clickhouse
Update docs for jdbc domain-compaction-threshold

Non-technical explanation

Allows pushdown of larger predicates into phoenix and clickhouse by default.

Release notes

( ) This is not user-visible and no release notes are required.
( ) Release notes are required, please propose a release note for me.
(x) Release notes are required, with the following suggested text:

# Phoenix
* Allow pushdown of larger predicates into Phoenix by default. ({issue}`14029`)

# ClickHouse
* Allow pushdown of larger predicates into ClickHouse by default. ({issue}`14029`)

Copy link
Copy Markdown
Member

@sopel39 sopel39 left a comment

Choose a reason for hiding this comment

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

lgtm % @mosabua doc review
% question about performance of large INs in phoenix/clickhouse

Copy link
Copy Markdown
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

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

Documentation looks good. The large range of default values is wild, but I assume thats a consequence of the different data sources.

@raunaqmorarka raunaqmorarka merged commit f25bd33 into trinodb:master Sep 9, 2022
@raunaqmorarka raunaqmorarka deleted the inc-domain branch September 9, 2022 08:20
@github-actions github-actions bot added this to the 396 milestone Sep 9, 2022
@lhofhansl
Copy link
Copy Markdown
Member

Thank you @raunaqmorarka !

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

Development

Successfully merging this pull request may close these issues.

4 participants