codecov-commenter commented on PR #18986:
URL: https://github.com/apache/hudi/pull/18986#issuecomment-4687615090

   ## 
[Codecov](https://app.codecov.io/gh/apache/hudi/pull/18986?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   :x: Patch coverage is `68.36735%` with `155 lines` in your changes missing 
coverage. Please review.
   :white_check_mark: Project coverage is 62.76%. Comparing base 
([`8933224`](https://app.codecov.io/gh/apache/hudi/commit/89332241ea0b42671dae81b5ef06cb7ab9fad64b?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache))
 to head 
([`f2b0791`](https://app.codecov.io/gh/apache/hudi/commit/f2b079109192e6b181858c964fdc8ab4909f4aae?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   :warning: Report is 10 commits behind head on master.
   
   | [Files with missing 
lines](https://app.codecov.io/gh/apache/hudi/pull/18986?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Patch % | Lines |
   |---|---|---|
   | 
[.../org/apache/hudi/hive/util/JDBCConnectionPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FJDBCConnectionPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9KREJDQ29ubmVjdGlvblBvb2wuamF2YQ==)
 | 51.48% | [46 Missing and 3 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[...java/org/apache/hudi/hive/util/HiveDriverPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FHiveDriverPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9IaXZlRHJpdmVyUG9vbC5qYXZh)
 | 74.19% | [29 Missing and 3 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[...in/java/org/apache/hudi/hive/ddl/JDBCExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FJDBCExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0pEQkNFeGVjdXRvci5qYXZh)
 | 49.05% | [21 Missing and 6 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[...rg/apache/hudi/hive/util/IMetaStoreClientPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FIMetaStoreClientPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9JTWV0YVN0b3JlQ2xpZW50UG9vbC5qYXZh)
 | 72.05% | [16 Missing and 3 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[...org/apache/hudi/hive/ddl/HiveQueryDDLExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FHiveQueryDDLExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0hpdmVRdWVyeURETEV4ZWN1dG9yLmphdmE=)
 | 54.16% | [6 Missing and 5 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[.../java/org/apache/hudi/hive/ddl/HMSDDLExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FHMSDDLExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0hNU0RETEV4ZWN1dG9yLmphdmE=)
 | 84.12% | [7 Missing and 3 partials :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2FHoodieHiveSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvSG9vZGllSGl2ZVN5bmNDbGllbnQuamF2YQ==)
 | 77.41% | [6 Missing and 1 partial :warning: 
](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   > :exclamation:  There is a different number of reports uploaded between 
BASE (8933224) and HEAD (f2b0791). Click for more details.
   > 
   > <details><summary>HEAD has 18 uploads less than BASE</summary>
   >
   >| Flag | BASE (8933224) | HEAD (f2b0791) |
   >|------|------|------|
   >|spark-java-tests|18|7|
   >|spark-scala-tests|12|5|
   ></details>
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@             Coverage Diff              @@
   ##             master   #18986      +/-   ##
   ============================================
   - Coverage     68.26%   62.76%   -5.51%     
   + Complexity    29513    27046    -2467     
   ============================================
     Files          2542     2545       +3     
     Lines        142627   142987     +360     
     Branches      17788    17850      +62     
   ============================================
   - Hits          97369    89743    -7626     
   - Misses        37253    45729    +8476     
   + Partials       8005     7515     -490     
   ```
   
   | 
[Flag](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Coverage Δ | |
   |---|---|---|
   | 
[common-and-other-modules](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `44.84% <68.36%> (+0.06%)` | :arrow_up: |
   | 
[hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `44.75% <ø> (+<0.01%)` | :arrow_up: |
   | 
[spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `48.06% <ø> (+0.01%)` | :arrow_up: |
   | 
[spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `41.60% <2.44%> (-7.18%)` | :arrow_down: |
   | 
[spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `32.57% <2.44%> (-12.28%)` | :arrow_down: |
   | 
[utilities](https://app.codecov.io/gh/apache/hudi/pull/18986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | `37.11% <5.71%> (-0.15%)` | :arrow_down: |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | [Files with missing 
lines](https://app.codecov.io/gh/apache/hudi/pull/18986?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Coverage Δ | |
   |---|---|---|
   | 
[...ava/org/apache/hudi/hive/HiveSyncConfigHolder.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2FHiveSyncConfigHolder.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvSGl2ZVN5bmNDb25maWdIb2xkZXIuamF2YQ==)
 | `99.21% <100.00%> (+0.08%)` | :arrow_up: |
   | 
[...rg/apache/hudi/hive/ddl/QueryBasedDDLExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FQueryBasedDDLExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL1F1ZXJ5QmFzZWRERExFeGVjdXRvci5qYXZh)
 | `87.12% <100.00%> (+1.07%)` | :arrow_up: |
   | 
[...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2FHoodieHiveSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvSG9vZGllSGl2ZVN5bmNDbGllbnQuamF2YQ==)
 | `50.50% <77.41%> (+1.57%)` | :arrow_up: |
   | 
[.../java/org/apache/hudi/hive/ddl/HMSDDLExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FHMSDDLExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0hNU0RETEV4ZWN1dG9yLmphdmE=)
 | `80.86% <84.12%> (+0.51%)` | :arrow_up: |
   | 
[...org/apache/hudi/hive/ddl/HiveQueryDDLExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FHiveQueryDDLExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0hpdmVRdWVyeURETEV4ZWN1dG9yLmphdmE=)
 | `62.63% <54.16%> (-3.04%)` | :arrow_down: |
   | 
[...rg/apache/hudi/hive/util/IMetaStoreClientPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FIMetaStoreClientPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9JTWV0YVN0b3JlQ2xpZW50UG9vbC5qYXZh)
 | `72.05% <72.05%> (ø)` | |
   | 
[...in/java/org/apache/hudi/hive/ddl/JDBCExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Fddl%2FJDBCExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvZGRsL0pEQkNFeGVjdXRvci5qYXZh)
 | `64.42% <49.05%> (-8.77%)` | :arrow_down: |
   | 
[...java/org/apache/hudi/hive/util/HiveDriverPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FHiveDriverPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9IaXZlRHJpdmVyUG9vbC5qYXZh)
 | `74.19% <74.19%> (ø)` | |
   | 
[.../org/apache/hudi/hive/util/JDBCConnectionPool.java](https://app.codecov.io/gh/apache/hudi/pull/18986?src=pr&el=tree&filepath=hudi-sync%2Fhudi-hive-sync%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fhive%2Futil%2FJDBCConnectionPool.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1zeW5jL2h1ZGktaGl2ZS1zeW5jL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL2hpdmUvdXRpbC9KREJDQ29ubmVjdGlvblBvb2wuamF2YQ==)
 | `51.48% <51.48%> (ø)` | |
   
   ... and [455 files with indirect coverage 
changes](https://app.codecov.io/gh/apache/hudi/pull/18986/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   </details>
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test 
Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, 
report on failures, and find test suite problems.
   </details>


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to