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.

Sign up for GitHub

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

Jump to bottom

[BACKPORT] Eliminate redundant eval node in optimization (#2683) #2688

Merged
merged 1 commit into from
Feb 9, 2022
Merged

[BACKPORT] Eliminate redundant eval node in optimization (#2683) #2688

merged 1 commit into from
Feb 9, 2022

Conversation

wjsi
Copy link
Member

@wjsi wjsi commented Feb 9, 2022

Backport of #2683

@wjsi wjsi added type: bug Something isn't working backport PR backported from pre-release branch to stable branch mod: optimization labels Feb 9, 2022
@wjsi wjsi added this to the v0.8.2 milestone Feb 9, 2022
@wjsi wjsi added this to In progress in DataFrame via automation Feb 9, 2022
qinxuye
qinxuye approved these changes Feb 9, 2022
View reviewed changes
Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM

DataFrame automation moved this from In progress to Reviewer approved Feb 9, 2022
@qinxuye qinxuye merged commit 2f9a081 into mars-project:v0.8 Feb 9, 2022
DataFrame automation moved this from Reviewer approved to Done Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qinxuye qinxuye qinxuye approved these changes

@hekaisheng hekaisheng Awaiting requested review from hekaisheng hekaisheng is a code owner

Assignees
No one assigned
Labels
backport PR backported from pre-release branch to stable branch mod: optimization type: bug Something isn't working
Projects
DataFrame
  
Done
Milestone
v0.8.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wjsi @qinxuye