codecov-commenter commented on PR #18986: URL: https://github.com/apache/hudi/pull/18986#issuecomment-4687608106
## [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.54%. 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 19 uploads less than BASE</summary> > >| Flag | BASE (8933224) | HEAD (f2b0791) | >|------|------|------| >|spark-java-tests|18|7| >|spark-scala-tests|12|4| ></details> <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## master #18986 +/- ## ============================================ - Coverage 68.26% 62.54% -5.73% + Complexity 29513 26901 -2612 ============================================ Files 2542 2545 +3 Lines 142627 142987 +360 Branches 17788 17850 +62 ============================================ - Hits 97369 89433 -7936 - Misses 37253 46145 +8892 + Partials 8005 7409 -596 ``` | [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.69% <2.44%> (-12.15%)` | :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 [456 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]
