Skip to content

[CALCITE-3852] RexSimplify doesn't simplify NOT EQUAL predicates#1854

Merged
chunweilei merged 1 commit intoapache:masterfrom
chunweilei:CALCITE-3852
Mar 18, 2020
Merged

[CALCITE-3852] RexSimplify doesn't simplify NOT EQUAL predicates#1854
chunweilei merged 1 commit intoapache:masterfrom
chunweilei:CALCITE-3852

Conversation

@chunweilei
Copy link
Copy Markdown
Contributor

No description provided.

@chunweilei chunweilei force-pushed the CALCITE-3852 branch 2 times, most recently from 611fc99 to 7ab36fd Compare March 13, 2020 09:44
Copy link
Copy Markdown
Member

@hsyuan hsyuan left a comment

Choose a reason for hiding this comment

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

+1

@hsyuan hsyuan added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Mar 17, 2020
Copy link
Copy Markdown
Member

@kgyrtkirk kgyrtkirk left a comment

Choose a reason for hiding this comment

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

sorry for the late notes @chunweilei ; I didn't got the notification about that you've update the patch...

Copy link
Copy Markdown
Member

@kgyrtkirk kgyrtkirk left a comment

Choose a reason for hiding this comment

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

+1

@chunweilei
Copy link
Copy Markdown
Contributor Author

Thank you for your review, @kgyrtkirk and @hsyuan .

@chunweilei chunweilei merged commit a8aa439 into apache:master Mar 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left. slow-tests-needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants