You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "voonhous (via GitHub)" <gi...@apache.org> on 2026/04/19 12:54:02 UTC
[PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
voonhous opened a new pull request, #18529:
URL: https://github.com/apache/hudi/pull/18529
### Describe the issue this Pull Request addresses
<!-- Either describe the issue inline here with motivation behind the changes
(or) link to an issue by including `Closes #<issue-number>` for context.
If this PR includes changes to the storage format, public APIs,
or has breaking changes, use `!` (e.g., feat!: ...) -->
`MapUtils.isNullOrEmpty(Map)` was byte-identical to `CollectionUtils.isNullOrEmpty(Map)`. Callers had drifted across both classes. Consolidate into `CollectionUtils` so there's one canonical utility.
### Summary and Changelog
<!-- Short, plain-English summary of what users gain or what changed in behavior.
Followed by a detailed log of all the changes. Highlight if any code was copied. -->
Internal refactor. No behavior change.
- Moved `containsAll(Map, Map)` into `CollectionUtils` and added `nonEmpty(Map)`.
- Deleted `MapUtils` and folded its test into `TestCollectionUtils`.
- Migrated 3 callers (`AWSGlueCatalogSyncClient`, `CleanPlanActionExecutor`, `HoodieHiveSyncClient`).
### Impact
<!-- Describe any public API or user-facing feature change or any performance impact. -->
None. Internal utility class, method bodies moved verbatim.
### Risk Level
<!-- Accepted values: none, low, medium or high. Other than `none`, explain the risk.
If medium or high, explain what verification was done to mitigate the risks. -->
low
### Documentation Update
<!-- Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".
- The config description must be updated if new configs are added or the default value of the configs are changed.
- Any new feature or user-facing change requires updating the Hudi website. Please follow the
[instruction](https://hudi.apache.org/contribute/developer-setup#website) to make changes to the website. -->
none
### Contributor's checklist
- [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
- [ ] Enough context is provided in the sections above
- [ ] Adequate tests were added if applicable
--
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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276026709
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 48.51%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 21 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|4|
>|spark-java-tests|15|2|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 48.51% -20.33%
+ Complexity 28323 16269 -12054
=============================================
Files 2467 1978 -489
Lines 135839 109965 -25874
Branches 16483 13751 -2732
=============================================
- Hits 93518 53352 -40166
- Misses 34922 51101 +16179
+ Partials 7399 5512 -1887
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.13% <14.28%> (-15.80%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.67% <14.28%> (-12.78%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `45.12% <50.00%> (-12.38%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1548 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276029784
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 54.29%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 18 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|3|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 54.29% -14.55%
+ Complexity 28323 18441 -9882
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 61119 -32399
- Misses 34922 45155 +10233
+ Partials 7399 6294 -1105
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.18% <14.28%> (-15.75%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `46.34% <100.00%> (-11.16%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1397 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "hudi-agent (via GitHub)" <gi...@apache.org>.
hudi-agent commented on code in PR #18529:
URL: https://github.com/apache/hudi/pull/18529#discussion_r3107457443
##########
hudi-common/src/main/java/org/apache/hudi/common/util/CollectionUtils.java:
##########
@@ -69,6 +69,14 @@ public static boolean nonEmpty(Collection<?> c) {
return !isNullOrEmpty(c);
}
+ public static boolean nonEmpty(Map<?, ?> m) {
+ return !isNullOrEmpty(m);
+ }
+
+ public static boolean containsAll(Map<?, ?> m1, Map<?, ?> m2) {
+ return m1.entrySet().containsAll(m2.entrySet());
Review Comment:
🤖 nit: `m1` and `m2` don't tell callers which side is the "container" — could you use something like `container` and `subset` (or add a brief Javadoc) so it's clear the semantics are "does m1 contain all entries of m2" without having to read the body?
<sub><i>- Generated by an AI agent and may contain mistakes. Please verify any suggestions before applying.</i></sub>
--
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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4275967250
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `50.00000%` with `1 line` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 44.77%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 28 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-client-hadoop-common|1|0|
>|spark-scala-tests|10|0|
>|spark-java-tests|15|0|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 44.77% -24.08%
+ Complexity 28323 8496 -19827
=============================================
Files 2467 1198 -1269
Lines 135839 62349 -73490
Branches 16483 6733 -9750
=============================================
- Hits 93518 27914 -65604
+ Misses 34922 31383 -3539
+ Partials 7399 3052 -4347
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `68.04% <ø> (-21.65%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `34.14% <50.00%> (-23.36%)` | :arrow_down: |
... and [2066 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276086063
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.85%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=========================================
Coverage 68.84% 68.85%
+ Complexity 28323 28321 -2
=========================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
=========================================
+ Hits 93518 93530 +12
+ Misses 34922 34908 -14
Partials 7399 7399
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.91% <28.57%> (-0.02%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [11 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276013433
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 46.68%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 26 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|1|
>|spark-java-tests|15|0|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 46.68% -22.17%
+ Complexity 28323 15025 -13298
=============================================
Files 2467 1943 -524
Lines 135839 105110 -30729
Branches 16483 12682 -3801
=============================================
- Hits 93518 49067 -44451
- Misses 34922 51004 +16082
+ Partials 7399 5039 -2360
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.08% <14.28%> (-12.37%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `40.24% <50.00%> (-17.26%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1636 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276041987
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 67.70%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 67.70% -1.15%
+ Complexity 28323 27798 -525
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 91962 -1556
- Misses 34922 36474 +1552
- Partials 7399 7401 +2
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.59% <14.28%> (-6.35%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.12% <42.85%> (-0.33%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [188 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276024267
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 48.51%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 22 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|3|
>|spark-java-tests|15|2|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 48.51% -20.34%
+ Complexity 28323 16269 -12054
=============================================
Files 2467 1978 -489
Lines 135839 109965 -25874
Branches 16483 13751 -2732
=============================================
- Hits 93518 53349 -40169
- Misses 34922 51103 +16181
+ Partials 7399 5513 -1886
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.13% <14.28%> (-15.80%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.67% <14.28%> (-12.78%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `45.12% <50.00%> (-12.38%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1548 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276014744
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 46.19%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 25 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|2|
>|spark-java-tests|15|0|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 46.19% -22.65%
+ Complexity 28323 15187 -13136
=============================================
Files 2467 1960 -507
Lines 135839 107599 -28240
Branches 16483 13241 -3242
=============================================
- Hits 93518 49710 -43808
- Misses 34922 52732 +17810
+ Partials 7399 5157 -2242
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.92% <14.28%> (-12.53%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `40.24% <50.00%> (-17.26%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1607 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276043235
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 67.81%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 67.81% -1.03%
+ Complexity 28323 27865 -458
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92121 -1397
- Misses 34922 36268 +1346
- Partials 7399 7448 +49
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.59% <14.28%> (-6.35%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.30% <42.85%> (-0.15%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [174 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4275952054
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"status" : "PENDING",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=13367",
"triggerID" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* da76458c933bda5d26395d5cd81c24c9d0779a68 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=13367)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276003091
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `50.00000%` with `1 line` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 53.88%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 27 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|0|
>|spark-java-tests|15|0|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 53.88% -14.96%
+ Complexity 28323 12310 -16013
=============================================
Files 2467 1428 -1039
Lines 135839 71361 -64478
Branches 16483 8068 -8415
=============================================
- Hits 93518 38456 -55062
+ Misses 34922 29463 -5459
+ Partials 7399 3442 -3957
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `36.58% <50.00%> (-20.92%)` | :arrow_down: |
... and [1813 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276035159
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 54.36%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 16 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|5|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 54.36% -14.49%
+ Complexity 28323 18469 -9854
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 61195 -32323
- Misses 34922 45092 +10170
+ Partials 7399 6281 -1118
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.41% <14.28%> (-16.53%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `46.34% <100.00%> (-11.16%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1395 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276018074
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 45.65%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 24 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|3|
>|spark-java-tests|15|0|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 45.65% -23.19%
+ Complexity 28323 15350 -12973
=============================================
Files 2467 1978 -489
Lines 135839 109960 -25879
Branches 16483 13745 -2738
=============================================
- Hits 93518 50206 -43312
- Misses 34922 54452 +19530
+ Partials 7399 5302 -2097
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.67% <14.28%> (-12.78%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `40.24% <50.00%> (-17.26%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1597 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276028799
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 54.29%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 18 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|3|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 54.29% -14.55%
+ Complexity 28323 18441 -9882
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 61119 -32399
- Misses 34922 45155 +10233
+ Partials 7399 6294 -1105
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.18% <14.28%> (-15.75%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `46.34% <100.00%> (-11.16%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1397 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276050354
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 67.90%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 67.90% -0.94%
+ Complexity 28323 27909 -414
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92240 -1278
- Misses 34922 36168 +1246
- Partials 7399 7429 +30
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.22% <28.57%> (-3.71%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.30% <42.85%> (-0.15%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [147 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276066947
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.84%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=========================================
Coverage 68.84% 68.84%
+ Complexity 28323 28321 -2
=========================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
=========================================
+ Hits 93518 93521 +3
+ Misses 34922 34917 -5
Partials 7399 7399
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.88% <28.57%> (-0.06%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [11 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276060087
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.06%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 68.06% -0.78%
+ Complexity 28323 28015 -308
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92463 -1055
- Misses 34922 35916 +994
- Partials 7399 7458 +59
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.35% <28.57%> (-3.58%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [132 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276047951
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 67.90%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 67.90% -0.95%
+ Complexity 28323 27909 -414
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92238 -1280
- Misses 34922 36168 +1246
- Partials 7399 7431 +32
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.22% <28.57%> (-3.71%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.30% <42.85%> (-0.15%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [148 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276020746
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `14.28571%` with `6 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 48.47%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | 50.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 23 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|3|
>|spark-java-tests|15|1|
>|common-and-other-modules|1|0|
>|utilities|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 48.47% -20.37%
+ Complexity 28323 16259 -12064
=============================================
Files 2467 1978 -489
Lines 135839 109960 -25879
Branches 16483 13745 -2738
=============================================
- Hits 93518 53307 -40211
- Misses 34922 51134 +16212
+ Partials 7399 5519 -1880
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.48% <14.28%> (-15.45%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.67% <14.28%> (-12.78%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `45.12% <50.00%> (-12.38%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `0.00% <0.00%> (-48.93%)` | :arrow_down: |
... and [1550 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "danny0405 (via GitHub)" <gi...@apache.org>.
danny0405 merged PR #18529:
URL: https://github.com/apache/hudi/pull/18529
--
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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276052917
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.00%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 68.00% -0.85%
+ Complexity 28323 27970 -353
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92371 -1147
- Misses 34922 35993 +1071
- Partials 7399 7473 +74
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.22% <28.57%> (-3.71%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [143 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4275947890
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* da76458c933bda5d26395d5cd81c24c9d0779a68 UNKNOWN
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276038004
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `42.85714%` with `4 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 63.31%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 25.00% | [0 Missing and 3 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 14 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|6|
>|spark-java-tests|15|6|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 63.31% -5.53%
+ Complexity 28323 21567 -6756
============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
============================================
- Hits 93518 71276 -22242
+ Misses 34922 34570 -352
+ Partials 7399 6722 -677
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.59% <14.28%> (-6.35%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.91% <42.85%> (-0.54%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `54.87% <100.00%> (-2.63%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `27.15% <25.00%> (-21.78%)` | :arrow_down: |
... and [1106 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276031940
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 54.32%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 17 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|4|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 54.32% -14.52%
+ Complexity 28323 18455 -9868
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 61158 -32360
- Misses 34922 45124 +10202
+ Partials 7399 6286 -1113
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.80% <14.28%> (-16.14%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `46.34% <100.00%> (-11.16%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1396 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276047325
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 67.83%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 67.83% -1.01%
+ Complexity 28323 27872 -451
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 92151 -1367
- Misses 34922 36239 +1317
- Partials 7399 7447 +48
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.65% <14.28%> (-6.29%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.30% <42.85%> (-0.15%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [168 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276028583
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 54.29%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 18 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|3|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 54.29% -14.55%
+ Complexity 28323 18441 -9882
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 61119 -32399
- Misses 34922 45155 +10233
+ Partials 7399 6294 -1105
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.18% <14.28%> (-15.75%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `46.34% <100.00%> (-11.16%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1397 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276036534
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `28.57143%` with `5 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 57.94%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | 0.00% | [0 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?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 (eaaae8a) and HEAD (da76458). Click for more details.
>
> <details><summary>HEAD has 15 uploads less than BASE</summary>
>
>| Flag | BASE (eaaae8a) | HEAD (da76458) |
>|------|------|------|
>|spark-scala-tests|10|5|
>|spark-java-tests|15|6|
>|common-and-other-modules|1|0|
></details>
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
=============================================
- Coverage 68.84% 57.94% -10.90%
+ Complexity 28323 19714 -8609
=============================================
Files 2467 2007 -460
Lines 135839 112568 -23271
Branches 16483 14305 -2178
=============================================
- Hits 93518 65230 -28288
- Misses 34922 41052 +6130
+ Partials 7399 6286 -1113
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `42.59% <14.28%> (-6.35%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `32.55% <14.28%> (-12.90%)` | :arrow_down: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `88.65% <ø> (-1.04%)` | :arrow_down: |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `50.00% <100.00%> (-7.50%)` | :arrow_down: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `0.00% <0.00%> (-49.69%)` | :arrow_down: |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `23.65% <0.00%> (-25.27%)` | :arrow_down: |
... and [1265 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276063357
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.84%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
- Coverage 68.84% 68.84% -0.01%
+ Complexity 28323 28319 -4
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
- Hits 93518 93511 -7
- Misses 34922 34927 +5
Partials 7399 7399
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.80% <28.57%> (-0.13%)` | :arrow_down: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [11 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276097040
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.85%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
+ Coverage 68.84% 68.85% +0.01%
+ Complexity 28323 28322 -1
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
+ Hits 93518 93531 +13
+ Misses 34922 34907 -15
Partials 7399 7399
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.95% <28.57%> (+0.02%)` | :arrow_up: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [12 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276087596
## [Codecov](https://app.codecov.io/gh/apache/hudi/pull/18529?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 `57.14286%` with `3 lines` in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 68.85%. Comparing base ([`eaaae8a`](https://app.codecov.io/gh/apache/hudi/commit/eaaae8a4f9abf57f3b06da0420d37a084ebfa3e5?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`da76458`](https://app.codecov.io/gh/apache/hudi/commit/da76458c933bda5d26395d5cd81c24c9d0779a68?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
| [Files with missing lines](https://app.codecov.io/gh/apache/hudi/pull/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
|---|---|---|
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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.00% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | 0.00% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #18529 +/- ##
============================================
+ Coverage 68.84% 68.85% +0.01%
+ Complexity 28323 28322 -1
============================================
Files 2467 2466 -1
Lines 135839 135837 -2
Branches 16483 16483
============================================
+ Hits 93518 93531 +13
+ Misses 34922 34907 -15
Partials 7399 7399
```
| [Flag](https://app.codecov.io/gh/apache/hudi/pull/18529/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/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.65% <57.14%> (+<0.01%)` | :arrow_up: |
| [hadoop-mr-java-client](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `44.77% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-client-hadoop-common](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.40% <50.00%> (-0.01%)` | :arrow_down: |
| [spark-java-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `48.95% <28.57%> (+0.01%)` | :arrow_up: |
| [spark-scala-tests](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `45.45% <42.85%> (+<0.01%)` | :arrow_up: |
| [utilities](https://app.codecov.io/gh/apache/hudi/pull/18529/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.18% <28.57%> (+<0.01%)` | :arrow_up: |
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/18529?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [...di/table/action/clean/CleanPlanActionExecutor.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-client%2Fhudi-client-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Ftable%2Faction%2Fclean%2FCleanPlanActionExecutor.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jbGllbnQvaHVkaS1jbGllbnQtY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9odWRpL3RhYmxlL2FjdGlvbi9jbGVhbi9DbGVhblBsYW5BY3Rpb25FeGVjdXRvci5qYXZh) | `89.69% <ø> (ø)` | |
| [...a/org/apache/hudi/common/util/CollectionUtils.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Fcommon%2Futil%2FCollectionUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvY29tbW9uL3V0aWwvQ29sbGVjdGlvblV0aWxzLmphdmE=) | `59.75% <100.00%> (+2.25%)` | :arrow_up: |
| [...apache/hudi/aws/sync/AWSGlueCatalogSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?src=pr&el=tree&filepath=hudi-aws%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fhudi%2Faws%2Fsync%2FAWSGlueCatalogSyncClient.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-aHVkaS1hd3Mvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2h1ZGkvYXdzL3N5bmMvQVdTR2x1ZUNhdGFsb2dTeW5jQ2xpZW50LmphdmE=) | `49.68% <0.00%> (ø)` | |
| [...ava/org/apache/hudi/hive/HoodieHiveSyncClient.java](https://app.codecov.io/gh/apache/hudi/pull/18529?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==) | `48.92% <50.00%> (ø)` | |
... and [12 files with indirect coverage changes](https://app.codecov.io/gh/apache/hudi/pull/18529/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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] chore(common): Consolidate MapUtils into CollectionUtils [hudi]
Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #18529:
URL: https://github.com/apache/hudi/pull/18529#issuecomment-4276108139
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=13367",
"triggerID" : "da76458c933bda5d26395d5cd81c24c9d0779a68",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* da76458c933bda5d26395d5cd81c24c9d0779a68 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=13367)
<details>
<summary>Bot commands</summary>
@hudi-bot supports the following commands:
- `@hudi-bot run azure` re-run the last Azure build
</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: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org