Skip to content

feat(spark): map from EqualNullSafe fn to is_not_distinct_from#306

Merged
vbarua merged 1 commit intosubstrait-io:mainfrom
andrew-coleman:not_distinct
Oct 16, 2024
Merged

feat(spark): map from EqualNullSafe fn to is_not_distinct_from#306
vbarua merged 1 commit intosubstrait-io:mainfrom
andrew-coleman:not_distinct

Conversation

@andrew-coleman
Copy link
Copy Markdown
Member

Add support for the <=> operator (NULL safe equals) which gets generated by the spark optimiser for certain queries.

Enables three more successful TPC-DS tests.

@vbarua vbarua changed the title feat(spark): support NULL safe equality operator feat(spark): map from EqualNullSafe fn to is_not_distinct_from Oct 16, 2024
Copy link
Copy Markdown
Member

@vbarua vbarua left a comment

Choose a reason for hiding this comment

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

LGTM

s[GreaterThan]("gt"),
s[GreaterThanOrEqual]("gte"),
s[EqualTo]("equal"),
s[EqualNullSafe]("is_not_distinct_from"),
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.

Add support for the <=> operator (NULL safe equals) which gets
generated by the spark optimiser for certain queries.

Enables three more successful TPC-DS tests.

Signed-off-by: Andrew Coleman <andrew_coleman@uk.ibm.com>
@vbarua vbarua merged commit 357cc01 into substrait-io:main Oct 16, 2024
@andrew-coleman andrew-coleman deleted the not_distinct branch October 17, 2024 08:28
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.

2 participants