Skip to content

Push varchar non-equality comparisons into PostgreSQL#13894

Closed
findepi wants to merge 1 commit intotrinodb:masterfrom
findepi:findepi/push-varchar-non-equality-comparisons-into-postgresql-675cde
Closed

Push varchar non-equality comparisons into PostgreSQL#13894
findepi wants to merge 1 commit intotrinodb:masterfrom
findepi:findepi/push-varchar-non-equality-comparisons-into-postgresql-675cde

Conversation

@findepi
Copy link
Copy Markdown
Member

@findepi findepi commented Aug 29, 2022

This enables by default an existing feature.

Follows #9746
Relates to #7496

This enables by default an existing feature.
@findepi findepi requested review from ebyhr, hashhar and takezoe August 29, 2022 09:29
@cla-bot cla-bot bot added the cla-signed label Aug 29, 2022
}

@Config("postgresql.experimental.enable-string-pushdown-with-collate")
@Config("postgresql.enable-string-pushdown-with-collate")
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.

Does it require docs update in this PR or in a follow-up?

Copy link
Copy Markdown
Member

@hashhar hashhar left a comment

Choose a reason for hiding this comment

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

It'll currently degrade performance for equality as well.

See CollationAwareQueryBuilder. We should probably make CollationAwareQueryBuilder use collations only for inequalities to improve it.

@findepi
Copy link
Copy Markdown
Member Author

findepi commented Aug 29, 2022

It'll currently degrade performance for equality as well.

Good point. Do we want this to be documented?

See CollationAwareQueryBuilder. We should probably make CollationAwareQueryBuilder use collations only for inequalities to improve it.

Good idea. Also, some values should be easy to compare regardless of case-sensitivity (I'm consciously not considering "arbitrary collections"). See #13896.

@findepi findepi marked this pull request as draft August 29, 2022 10:22
@findepi
Copy link
Copy Markdown
Member Author

findepi commented Aug 30, 2022

It'll currently degrade performance for equality as well.

Let's document this.

@findepi findepi closed this Aug 30, 2022
@findepi findepi deleted the findepi/push-varchar-non-equality-comparisons-into-postgresql-675cde branch August 30, 2022 13:49
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.

3 participants