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

Skip details of shuffled chunks in meta #2600

Merged
merged 1 commit into from
Dec 7, 2021
Merged

Skip details of shuffled chunks in meta #2600

merged 1 commit into from
Dec 7, 2021

Conversation

wjsi
Copy link
Member

@wjsi wjsi commented Dec 3, 2021 •
edited
Loading

What do these changes do?

Skip details of shuffled chunks in meta to jmprove performance.

Related issue number

Fixes #xxxx

Check code requirements

  • tests added / passed (if needed)
  • Ensure all linting tests pass, see here for how to run them

@wjsi wjsi added type: enhancement request mod: dataframe to be backported Indicate that the PR need to be backported to stable branch labels Dec 3, 2021
@wjsi wjsi added this to the v0.9.0a1 milestone Dec 3, 2021
@wjsi wjsi requested a review from qinxuye as a code owner December 3, 2021 06:40
hekaisheng
hekaisheng approved these changes Dec 7, 2021
View reviewed changes
Copy link
Contributor

@hekaisheng hekaisheng left a comment

Choose a reason for hiding this comment

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

LGTM

qinxuye
qinxuye approved these changes Dec 7, 2021
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

@qinxuye qinxuye merged commit 49ce0c1 into mars-project:master Dec 7, 2021
@wjsi wjsi deleted the enh/simplify_shuffle_meta branch December 7, 2021 08:27
wjsi added a commit to wjsi/mars that referenced this pull request Dec 7, 2021
@wjsi
Skip details of shuffled chunks in meta (mars-project#2600)
f3beb9d
@wjsi wjsi mentioned this pull request Dec 7, 2021
Merged
hekaisheng pushed a commit that referenced this pull request Dec 8, 2021
@wjsi
[BACKPORT] Skip details of shuffled chunks in meta (#2600) (#2609)
bcca2ad
@wjsi wjsi added backported already PR has been backported and removed to be backported Indicate that the PR need to be backported to stable branch labels Dec 8, 2021
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 hekaisheng approved these changes

Assignees
No one assigned
Labels
backported already PR has been backported mod: dataframe type: enhancement request
Projects
None yet
Milestone
v0.9.0a1
Development

Successfully merging this pull request may close these issues.
3 participants
@wjsi @qinxuye @hekaisheng