-
Notifications
You must be signed in to change notification settings - Fork 425
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
TEZ-3268: Provide a demuxer sample app that uses fair routing #320
base: master
Are you sure you want to change the base?
Conversation
.addEdge(Edge.create(inputVertex, demuxVertex, edgeConf.createDefaultEdgeProperty())); | ||
} | ||
|
||
private DAG createDagWithUnion(TezConfiguration tezConf, List<Path> inputPaths, Path outputPath, |
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.
Note: This case doesn't work yet until we merge #306
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.
#306 was closed due to some reasons
is this PR suitable to be adapted to work without that patch? if so, still interested in seeing this in the codebase
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.
@abstractdog Thanks for asking. I closed #306 and still need to implement an alternative solution. At this point, I removed the case using UNION from the sample.
9e09665
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
This is a sample application that uses FairShuffleVertexManager.
We can use it like this.
The following are the results on my machine. We can see skewed keys are written by multiple reducers.