-
Notifications
You must be signed in to change notification settings - Fork 141
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
[#855] feat(tez): Support Tez Output UnorderedPartitionedKVOutput #943
Conversation
Codecov Report
@@ Coverage Diff @@
## master #943 +/- ##
============================================
+ Coverage 55.93% 57.85% +1.91%
+ Complexity 2336 2208 -128
============================================
Files 349 318 -31
Lines 17578 14139 -3439
Branches 1394 1281 -113
============================================
- Hits 9833 8180 -1653
+ Misses 7177 5485 -1692
+ Partials 568 474 -94
... and 34 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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, thanks @bin41215 , merged to master.
What changes were proposed in this pull request?
support tez write UnorderedPartitionedKVOutput
Why are the changes needed?
Fix: #855
Does this PR introduce any user-facing change?
No.
How was this patch tested?
UT