I just see that HIVE-28519: Upgrade Maven SureFire Plugin to latest version 3.5.1 is trying to fix the issue. And the Jenkins shows that it works https://ci.hive.apache.org/job/hive-precommit/view/change-requests/job/PR-5530/lastCompletedBuild/testReport/ .
I haven't reviewed this code carefully. I see that Ayush Saxena has did some comments. Therefore, should we merge HIVE-28519 to fix the issue? Or we should continue to revert HIVE-28417? I don't want to block commits progress. If HIVE-28519 is ready, I prefer to merge it quickly to fix the skipped tests. Thanks, Butao Zhang ---- Replied Message ---- | From | Denys Kuzmenko<dkuzme...@apache.org> | | Date | 11/6/2024 17:55 | | To | <dev@hive.apache.org> | | Subject | Re: Pause commits to master due to HIVE-28611 | https://ci.hive.apache.org/job/hive-precommit/job/master/2323/testReport/ is the first build we got -60k tests. Seems related to HIVE-28417: Bump Log4j2 to 2.24.1 to facilitate compilation of GraalVM Native Image We should be more careful with CVEs and dependency upgrades, they don't bring much business value, but could break the product. PS: Maybe we should consider adding some test count checks.
