Skip to content
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

executor: refine the usage of expression blacklist (#11940) #11944

Closed
wants to merge 4 commits into from

Conversation

sre-bot
Copy link
Contributor

@sre-bot sre-bot commented Aug 30, 2019

cherry-pick #11940 to release-3.0


What problem does this PR solve?

refine the usage of expression blacklist

What is changed and how it works?

Before this commit, we need to use an alias name to avoid the pushing down of a specific expression.
After this commit, we can use the original name of expression to avoid it.

Check List

Tests
exist tests

Code changes

  • Has exported function/method change
  • Has exported variable/fields change

Side effects

N/A

Related changes

  • Need to cherry-pick to the release branch

Release note

  • Write release note for bug-fix or new feature.

@sre-bot
Copy link
Contributor Author

sre-bot commented Aug 30, 2019

/run-all-tests

@lzmhhh123
Copy link
Contributor

@XuHuaiyu PTAL.

@XuHuaiyu
Copy link
Contributor

XuHuaiyu commented Sep 2, 2019

dup with #11975

@XuHuaiyu XuHuaiyu closed this Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/execution SIG execution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants