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/21 06:51:26 UTC

[GitHub] [hudi] codecov-commenter edited a comment on pull request #1149: [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/b71f25f210c4004a2dcc97a9967399e74f870fc7&el=desc) will **increase** coverage by `9.65%`.
   > The diff coverage is `52.50%`.
   
   [![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     46.45%   56.10%   +9.65%     
   - Complexity     2811     3498     +687     
   ============================================
     Files           442      454      +12     
     Lines         19131    19373     +242     
     Branches       1931     1942      +11     
   ============================================
   + Hits           8888    10870    +1982     
   + Misses         9483     7771    -1712     
   + Partials        760      732      -28     
   ```
   
   | Flag | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | #hudicli | `68.65% <52.50%> (+38.81%)` | `1489.00 <20.00> (+919.00)` | |
   | #hudiclient | `79.01% <52.50%> (+52.40%)` | `1317.00 <20.00> (+919.00)` | |
   | #hudicommon | `54.81% <ø> (+0.02%)` | `1508.00 <ø> (-1.00)` | :arrow_up: |
   | #hudihadoopmr | `38.99% <ø> (ø)` | `163.00 <ø> (ø)` | |
   | #hudihivesync | `72.25% <ø> (ø)` | `121.00 <ø> (ø)` | |
   | #hudispark | `49.24% <ø> (ø)` | `122.00 <ø> (ø)` | |
   | #huditimelineservice | `63.47% <ø> (ø)` | `47.00 <ø> (ø)` | |
   | #hudiutilities | `12.07% <ø> (-62.58%)` | `48.00 <ø> (-231.00)` | |
   
   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> (?)` | |
   | [...che/hudi/table/action/commit/BulkInsertHelper.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvdGFibGUvYWN0aW9uL2NvbW1pdC9CdWxrSW5zZXJ0SGVscGVyLmphdmE=) | `74.19% <60.00%> (+74.19%)` | `3.00 <1.00> (+3.00)` | |
   | [...java/org/apache/hudi/config/HoodieWriteConfig.java](https://codecov.io/gh/apache/hudi/pull/1149/diff?src=pr&el=tree#diff-aHVkaS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29uZmlnL0hvb2RpZVdyaXRlQ29uZmlnLmphdmE=) | `79.32% <83.33%> (+35.84%)` | `90.00 <1.00> (+33.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%> (+6.86%)` | `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 [157 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