-
Notifications
You must be signed in to change notification settings - Fork 325
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
[Shuffle] Add n_mappers
and n_reducers
to ShuffleProxy
#3160
[Shuffle] Add n_mappers
and n_reducers
to ShuffleProxy
#3160
Conversation
3ecd840
to
aa3e152
Compare
In my opinion, |
Some reducer chunks are not included in the chunk graph, we can't get all info when build graph. |
aa3e152
to
f934100
Compare
f934100
to
982cb9b
Compare
982cb9b
to
ca8055e
Compare
e8fc3a4
to
599d60b
Compare
7fe1a25
to
c0bc3a7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
n_mappers
and n_reducers
to ShuffleProxy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
#3160) (#3169) Co-authored-by: Shawn <[email protected]>
What do these changes do?
n_mappers/n_reducers
info to shuffle proxy, so it can be used for shuffle optimization such as fetch by index.n_reducers
/reducer_ordinal
by set those attributes inMapReduceOperand#new_chunks
Related issue number
Fixes #xxxx
Check code requirements