New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-5524] [table] Support early out for code generated AND/OR condition #3372

Closed
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@KurtYoung
Contributor

KurtYoung commented Feb 21, 2017

For condition like a AND b, if the result of a is false, we can save b from execution.

For condition like a OR b, if the result of a is true, we can also save b from execution.

@KurtYoung

This comment has been minimized.

Show comment
Hide comment
@KurtYoung

KurtYoung Feb 27, 2017

Contributor

hey @twalthr, any more comments or this can be merged

Contributor

KurtYoung commented Feb 27, 2017

hey @twalthr, any more comments or this can be merged

@twalthr

This comment has been minimized.

Show comment
Hide comment
@twalthr

twalthr Mar 2, 2017

Contributor

Thanks for the update @KurtYoung. The code looks good. I will merge this.

Contributor

twalthr commented Mar 2, 2017

Thanks for the update @KurtYoung. The code looks good. I will merge this.

@asfgit asfgit closed this in f6c9b32 Mar 2, 2017

zentol added a commit to zentol/flink that referenced this pull request Mar 2, 2017

@KurtYoung KurtYoung deleted the KurtYoung:flink-5524 branch Mar 3, 2017

heytitle added a commit to heytitle/flink that referenced this pull request Apr 16, 2017

hequn8128 pushed a commit to hequn8128/flink that referenced this pull request Jun 22, 2017

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