Skip to content

[CALCITE-2671] GeodeFilter change to tansform Multiple Or s to IN SET…#942

Closed
chadasa wants to merge 1 commit intoapache:masterfrom
chadasa:calcite-geode-in-set-fix
Closed

[CALCITE-2671] GeodeFilter change to tansform Multiple Or s to IN SET…#942
chadasa wants to merge 1 commit intoapache:masterfrom
chadasa:calcite-geode-in-set-fix

Conversation

@chadasa
Copy link
Contributor

@chadasa chadasa commented Nov 26, 2018

…(Sandeep Chada)

Geode IN SET gives much higher performance than hundreds of OR statements.

…T(Sandeep Chada)

Geode IN SET gives much higher performance than hundreds of OR statements.
Also added bug fixes to support search by DATE, TIME, TIMESTAMP fields

foo = 1 or foo = 2 or foo = 3
-- equivalent to (but much faster in geode)
foo in SET(1, 2, 3)
@asfgit asfgit closed this in da57c90 Nov 28, 2018
F21 pushed a commit to F21/calcite that referenced this pull request Jan 3, 2019
…nto single IN SET. (Sandeep Chada)

Geode IN SET operator has better performance than multiple ORs (even if they're equivalent)

foo = 1 or foo = 2 or foo = 3
-- equivalent to (but much faster in geode)
foo in SET(1, 2, 3)

closes apache#942
wangxlong pushed a commit to wangxlong/calcite that referenced this pull request Feb 13, 2020
…nto single IN SET. (Sandeep Chada)

Geode IN SET operator has better performance than multiple ORs (even if they're equivalent)

foo = 1 or foo = 2 or foo = 3
-- equivalent to (but much faster in geode)
foo in SET(1, 2, 3)

closes apache#942
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Aug 28, 2025
…nto single IN SET. (Sandeep Chada)

Geode IN SET operator has better performance than multiple ORs (even if they're equivalent)

foo = 1 or foo = 2 or foo = 3
-- equivalent to (but much faster in geode)
foo in SET(1, 2, 3)

closes apache#942

Change-Id: Ib0e60b666c9ce80ea5e8e0902ef368dc8b29c54d
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.

1 participant