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] Fix df.loc[:] to make sure same index_value key generated (#2643) #2654

Merged
merged 1 commit into from
Jan 25, 2022
Merged

[BACKPORT] Fix df.loc[:] to make sure same index_value key generated (#2643) #2654

merged 1 commit into from
Jan 25, 2022

Conversation

wjsi
Copy link
Member

@wjsi wjsi commented Jan 24, 2022

Backport of #2643

@wjsi wjsi added type: bug Something isn't working mod: dataframe backport PR backported from pre-release branch to stable branch labels Jan 24, 2022
@wjsi wjsi added this to the v0.8.1 milestone Jan 24, 2022
@wjsi wjsi added this to In progress in DataFrame via automation Jan 24, 2022
qinxuye
qinxuye approved these changes Jan 25, 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 Jan 25, 2022
@qinxuye qinxuye merged commit a53cfab into mars-project:v0.8 Jan 25, 2022
DataFrame automation moved this from Reviewer approved to Done Jan 25, 2022
@wjsi wjsi deleted the backport/2643 branch January 25, 2022 03:32
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: dataframe type: bug Something isn't working
Projects
DataFrame
  
Done
Milestone
v0.8.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wjsi @qinxuye