You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2020/07/20 00:37:30 UTC

[GitHub] [hudi] codecov-commenter edited a comment on pull request #1149: [WIP] [HUDI-472] Introduce configurations and new modes of sorting for bulk_insert

codecov-commenter edited a comment on pull request #1149:
URL: https://github.com/apache/hudi/pull/1149#issuecomment-652734921


   # [Codecov](https://codecov.io/gh/apache/hudi/pull/1149?src=pr&el=h1) Report
   > Merging [#1149](https://codecov.io/gh/apache/hudi/pull/1149?src=pr&el=desc) into [master](https://codecov.io/gh/apache/hudi/commit/2be924fd3a04e2106f1ad3f9ce91890ea58c8a88&el=desc) will **increase** coverage by `0.27%`.
   > The diff coverage is `50.38%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/hudi/pull/1149/graphs/tree.svg?width=650&height=150&src=pr&token=VTTXabwbs2)](https://codecov.io/gh/apache/hudi/pull/1149?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #1149      +/-   ##
   ============================================
   + Coverage     60.38%   60.66%   +0.27%     
   - Complexity     3605     3686      +81     
   ============================================
     Files           436      450      +14     
     Lines         18949    19236     +287     
     Branches       1914     1933      +19     
   ============================================
   + Hits          11442    11669     +227     
   - Misses         6719     6768      +49     
   - Partials        788      799      +11     
   ```
   
   | Flag | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | #hudicli | `68.06% <50.38%> (-0.29%)` | `1445.00 <21.00> (+19.00)` | :arrow_down: |
   | #hudiclient | `78.51% <50.38%> (-0.66%)` | `1273.00 <21.00> (+19.00)` | :arrow_down: |
   | #hudicommon | `54.78% <ø> (+0.50%)` | `1509.00 <ø> (+24.00)` | |
   | #hudihadoopmr | `38.99% <ø> (-0.37%)` | `163.00 <ø> (ø)` | |
   | #hudihivesync | `72.25% <ø> (ø)` | `121.00 <ø> (ø)` | |
   | #hudispark | `49.24% <ø> (+5.01%)` | `122.00 <ø> (+46.00)` | |
   | #huditimelineservice | `63.47% <ø> (ø)` | `47.00 <ø> (ø)` | |
   | #hudiutilities | `74.64% <ø> (+0.89%)` | `279.00 <ø> (-8.00)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   | [Impacted Files](https://codecov.io/gh/apache/hudi/pull/1149?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [...sert/BulkInsertMapFunctionForNonSortedRecords.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvQnVsa0luc2VydE1hcEZ1bmN0aW9uRm9yTm9uU29ydGVkUmVjb3Jkcy5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [.../hudi/execution/bulkinsert/NonSortPartitioner.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvTm9uU29ydFBhcnRpdGlvbmVyLmphdmE=) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [...n/bulkinsert/RDDPartitionLocalSortPartitioner.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvUkREUGFydGl0aW9uTG9jYWxTb3J0UGFydGl0aW9uZXIuamF2YQ==) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [...ution/bulkinsert/RDDPartitionRangePartitioner.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvUkREUGFydGl0aW9uUmFuZ2VQYXJ0aXRpb25lci5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | [...tion/bulkinsert/BulkInsertInternalPartitioner.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvQnVsa0luc2VydEludGVybmFsUGFydGl0aW9uZXIuamF2YQ==) | `53.84% <53.84%> (ø)` | `2.00 <2.00> (?)` | |
   | [...java/org/apache/hudi/config/HoodieWriteConfig.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29uZmlnL0hvb2RpZVdyaXRlQ29uZmlnLmphdmE=) | `79.08% <60.00%> (-0.76%)` | `90.00 <2.00> (+2.00)` | :arrow_down: |
   | [...che/hudi/table/action/commit/BulkInsertHelper.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvdGFibGUvYWN0aW9uL2NvbW1pdC9CdWxrSW5zZXJ0SGVscGVyLmphdmE=) | `75.00% <62.50%> (-10.72%)` | `3.00 <1.00> (-1.00)` | |
   | [...pache/hudi/execution/CopyOnWriteInsertHandler.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL0NvcHlPbldyaXRlSW5zZXJ0SGFuZGxlci5qYXZh) | `94.11% <94.11%> (ø)` | `8.00 <8.00> (?)` | |
   | [.../org/apache/hudi/execution/LazyInsertIterable.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL0xhenlJbnNlcnRJdGVyYWJsZS5qYXZh) | `79.41% <100.00%> (-0.99%)` | `8.00 <1.00> (ø)` | |
   | [...di/execution/bulkinsert/BulkInsertMapFunction.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvZXhlY3V0aW9uL2J1bGtpbnNlcnQvQnVsa0luc2VydE1hcEZ1bmN0aW9uLmphdmE=) | `75.00% <100.00%> (ø)` | `1.00 <1.00> (?)` | |
   | ... and [54 more](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree-more) | |
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org