From 6054b163d0ffeae63a6bb7daa879c7737bb227ec Mon Sep 17 00:00:00 2001 From: Kiran Velumuri Date: Wed, 17 Sep 2025 10:39:55 +0530 Subject: [PATCH] HIVE-29206: Remove commons-lang from hive-exec shaded jar --- accumulo-handler/pom.xml | 10 +++++ cli/pom.xml | 12 ++++++ common/pom.xml | 4 ++ druid-handler/pom.xml | 8 ++++ hbase-handler/pom.xml | 12 ++++++ hcatalog/core/pom.xml | 4 ++ iceberg/iceberg-handler/pom.xml | 18 +++++++++ itests/hcatalog-unit/pom.xml | 6 +++ itests/hive-blobstore/pom.xml | 18 +++++++++ itests/hive-jmh/pom.xml | 16 ++++++++ itests/hive-minikdc/pom.xml | 12 ++++++ itests/hive-unit-hadoop2/pom.xml | 8 ++++ itests/hive-unit/pom.xml | 39 +++++++++++++++++++ itests/qtest-accumulo/pom.xml | 26 +++++++++++++ itests/qtest-druid/pom.xml | 8 ++++ itests/qtest-iceberg/pom.xml | 24 ++++++++++++ itests/qtest-kudu/pom.xml | 22 +++++++++++ itests/qtest/pom.xml | 24 ++++++++++++ itests/util/pom.xml | 8 ++++ llap-client/pom.xml | 8 ++++ llap-common/pom.xml | 8 ++++ llap-ext-client/pom.xml | 8 ++++ llap-server/pom.xml | 8 ++++ llap-tez/pom.xml | 16 ++++++++ pom.xml | 7 ++++ ql/pom.xml | 27 ++++++++++++- service/pom.xml | 16 ++++++++ shims/0.23/pom.xml | 18 +++++++++ standalone-metastore/metastore-common/pom.xml | 23 ----------- standalone-metastore/metastore-server/pom.xml | 27 ++----------- .../metastore-benchmarks/pom.xml | 4 ++ standalone-metastore/pom.xml | 30 ++++++++++++++ streaming/pom.xml | 12 ++++++ 33 files changed, 444 insertions(+), 47 deletions(-) diff --git a/accumulo-handler/pom.xml b/accumulo-handler/pom.xml index dbc98f1d9950..f04ccd0a9e52 100644 --- a/accumulo-handler/pom.xml +++ b/accumulo-handler/pom.xml @@ -43,11 +43,21 @@ commons-collections commons-collections + + commons-lang + commons-lang + org.apache.accumulo accumulo-fate + + + commons-lang + commons-lang + + org.apache.accumulo diff --git a/cli/pom.xml b/cli/pom.xml index ebb0fbff8c52..0eb44799248b 100644 --- a/cli/pom.xml +++ b/cli/pom.xml @@ -145,11 +145,23 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez tez-dag test + + + commons-lang + commons-lang + + org.mockito diff --git a/common/pom.xml b/common/pom.xml index 66931be0c033..7283602cdcd3 100644 --- a/common/pom.xml +++ b/common/pom.xml @@ -241,6 +241,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + diff --git a/druid-handler/pom.xml b/druid-handler/pom.xml index cd5cfa6ae0ae..f82bbc30394e 100644 --- a/druid-handler/pom.xml +++ b/druid-handler/pom.xml @@ -106,6 +106,10 @@ com.fasterxml.jackson.core jackson-databind + + commons-lang + commons-lang + org.apache.druid druid-aws-common @@ -151,6 +155,10 @@ com.google.code.findbugs annotations + + commons-lang + commons-lang + true diff --git a/hbase-handler/pom.xml b/hbase-handler/pom.xml index c87d78b29418..053e800cdb10 100644 --- a/hbase-handler/pom.xml +++ b/hbase-handler/pom.xml @@ -261,11 +261,23 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez tez-dag test + + + commons-lang + commons-lang + + org.mockito diff --git a/hcatalog/core/pom.xml b/hcatalog/core/pom.xml index e6ae718e78c7..93073297a6b2 100644 --- a/hcatalog/core/pom.xml +++ b/hcatalog/core/pom.xml @@ -303,6 +303,10 @@ ch.qos.reload4j reload4j + + commons-lang + commons-lang + diff --git a/iceberg/iceberg-handler/pom.xml b/iceberg/iceberg-handler/pom.xml index a4ab0c6b79d9..d4a041d1ca2d 100644 --- a/iceberg/iceberg-handler/pom.xml +++ b/iceberg/iceberg-handler/pom.xml @@ -65,6 +65,12 @@ org.apache.calcite calcite-core test + + + commons-lang + commons-lang + + com.esotericsoftware @@ -80,11 +86,23 @@ org.apache.tez tez-dag test + + + commons-lang + commons-lang + + org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.hive diff --git a/itests/hcatalog-unit/pom.xml b/itests/hcatalog-unit/pom.xml index 854ba44e41c2..4b776924e16a 100644 --- a/itests/hcatalog-unit/pom.xml +++ b/itests/hcatalog-unit/pom.xml @@ -215,6 +215,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + com.sun.jersey diff --git a/itests/hive-blobstore/pom.xml b/itests/hive-blobstore/pom.xml index 7daefebc13cc..4989caf3c86f 100644 --- a/itests/hive-blobstore/pom.xml +++ b/itests/hive-blobstore/pom.xml @@ -211,6 +211,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -222,6 +226,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -232,6 +240,12 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez @@ -242,6 +256,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/hive-jmh/pom.xml b/itests/hive-jmh/pom.xml index f347db5e8e3c..08161e81f3a0 100644 --- a/itests/hive-jmh/pom.xml +++ b/itests/hive-jmh/pom.xml @@ -57,11 +57,21 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + org.apache.tez tez-runtime-internals + + + commons-lang + commons-lang + + org.apache.hive @@ -79,6 +89,12 @@ org.apache.tez tez-runtime-library + + + commons-lang + commons-lang + + org.apache.hadoop diff --git a/itests/hive-minikdc/pom.xml b/itests/hive-minikdc/pom.xml index d4e0578304b4..a83dcccaef08 100644 --- a/itests/hive-minikdc/pom.xml +++ b/itests/hive-minikdc/pom.xml @@ -41,6 +41,10 @@ ${apache-directory-server.version} test + + commons-lang + commons-lang + dom4j dom4j @@ -158,6 +162,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -275,6 +283,10 @@ ch.qos.reload4j reload4j + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/hive-unit-hadoop2/pom.xml b/itests/hive-unit-hadoop2/pom.xml index f1115f40ff36..6d162802f8c3 100644 --- a/itests/hive-unit-hadoop2/pom.xml +++ b/itests/hive-unit-hadoop2/pom.xml @@ -53,6 +53,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -242,6 +246,10 @@ ch.qos.reload4j reload4j + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/hive-unit/pom.xml b/itests/hive-unit/pom.xml index 66a109a5cf47..bddd2b9c1342 100644 --- a/itests/hive-unit/pom.xml +++ b/itests/hive-unit/pom.xml @@ -128,6 +128,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -396,6 +400,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -407,6 +415,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -422,6 +434,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -432,6 +448,12 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez @@ -442,6 +464,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -457,6 +483,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + org.apache.hadoop @@ -558,6 +590,13 @@ + + + commons-lang:commons-lang + + true + Banned commons-lang:commons-lang dependency/transitive dependency was found! + org.junit.jupiter:junit-jupiter diff --git a/itests/qtest-accumulo/pom.xml b/itests/qtest-accumulo/pom.xml index 40911d9b754b..233a5d975d3b 100644 --- a/itests/qtest-accumulo/pom.xml +++ b/itests/qtest-accumulo/pom.xml @@ -299,6 +299,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + org.apache.hadoop @@ -360,6 +366,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -371,6 +381,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -386,6 +400,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -401,6 +419,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -436,6 +458,10 @@ commons-beanutils commons-beanutils-core + + commons-lang + commons-lang + diff --git a/itests/qtest-druid/pom.xml b/itests/qtest-druid/pom.xml index 2e3b84cbf31a..fbf3385c5815 100644 --- a/itests/qtest-druid/pom.xml +++ b/itests/qtest-druid/pom.xml @@ -46,6 +46,10 @@ druid-server ${druid.version} + + commons-lang + commons-lang + jersey-server com.sun.jersey @@ -69,6 +73,10 @@ druid-services ${druid.version} + + commons-lang + commons-lang + jersey-server com.sun.jersey diff --git a/itests/qtest-iceberg/pom.xml b/itests/qtest-iceberg/pom.xml index 6ac62e8fa0b3..2557234a5494 100644 --- a/itests/qtest-iceberg/pom.xml +++ b/itests/qtest-iceberg/pom.xml @@ -292,6 +292,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + org.apache.hadoop @@ -382,6 +388,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -393,6 +403,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -403,6 +417,12 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez @@ -413,6 +433,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/qtest-kudu/pom.xml b/itests/qtest-kudu/pom.xml index 006c0e3610d8..75eb8e470ac4 100644 --- a/itests/qtest-kudu/pom.xml +++ b/itests/qtest-kudu/pom.xml @@ -260,6 +260,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + org.apache.hadoop @@ -290,6 +296,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -301,6 +311,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -316,6 +330,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -331,6 +349,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/qtest/pom.xml b/itests/qtest/pom.xml index 90ec4ccb01a1..a93e3b3a2d62 100644 --- a/itests/qtest/pom.xml +++ b/itests/qtest/pom.xml @@ -285,6 +285,12 @@ hadoop-yarn-server-tests test tests + + + commons-lang + commons-lang + + org.apache.hive @@ -381,6 +387,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -392,6 +402,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -402,6 +416,12 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez @@ -412,6 +432,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/itests/util/pom.xml b/itests/util/pom.xml index 2322e5ea63ca..b7c4e79e8e9a 100644 --- a/itests/util/pom.xml +++ b/itests/util/pom.xml @@ -35,6 +35,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -187,6 +191,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + diff --git a/llap-client/pom.xml b/llap-client/pom.xml index d0c24e5207eb..4d74acab67dc 100644 --- a/llap-client/pom.xml +++ b/llap-client/pom.xml @@ -201,6 +201,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -224,6 +228,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/llap-common/pom.xml b/llap-common/pom.xml index 05bf2d3086d7..c195f5304f59 100644 --- a/llap-common/pom.xml +++ b/llap-common/pom.xml @@ -123,6 +123,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -161,6 +165,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/llap-ext-client/pom.xml b/llap-ext-client/pom.xml index 8a1bf97f95c5..a912257c49af 100644 --- a/llap-ext-client/pom.xml +++ b/llap-ext-client/pom.xml @@ -98,6 +98,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -109,6 +113,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/llap-server/pom.xml b/llap-server/pom.xml index 082d9ff69e2b..0292d1d5b16d 100644 --- a/llap-server/pom.xml +++ b/llap-server/pom.xml @@ -193,6 +193,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -220,6 +224,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/llap-tez/pom.xml b/llap-tez/pom.xml index 8ecaf03f50a6..1b98c05297cc 100644 --- a/llap-tez/pom.xml +++ b/llap-tez/pom.xml @@ -142,6 +142,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -153,6 +157,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -168,6 +176,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -191,6 +203,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging diff --git a/pom.xml b/pom.xml index 88bc690f90a1..8a0b7b36c414 100644 --- a/pom.xml +++ b/pom.xml @@ -1716,6 +1716,13 @@ A banned license dependency was found! + + + commons-lang:commons-lang + + true + Banned commons-lang:commons-lang dependency/transitive dependency was found! + diff --git a/ql/pom.xml b/ql/pom.xml index 5927cc5feae6..1558c5a2ccfd 100644 --- a/ql/pom.xml +++ b/ql/pom.xml @@ -97,6 +97,12 @@ commons-configuration commons-configuration ${commons-configuration.version} + + + commons-lang + commons-lang + + org.apache.commons @@ -630,6 +636,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -665,6 +675,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -680,6 +694,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -720,6 +738,10 @@ tez-mapreduce true + + commons-lang + commons-lang + org.apache.hadoop hadoop-common @@ -767,6 +789,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -1057,7 +1083,6 @@ org.apache.thrift:libthrift org.apache.thrift:libfb303 org.datanucleus:javax.jdo - commons-lang:commons-lang org.apache.commons:commons-lang3 org.jodd:jodd-util com.tdunning:json diff --git a/service/pom.xml b/service/pom.xml index 5921d2350c77..d7c60ca75de3 100644 --- a/service/pom.xml +++ b/service/pom.xml @@ -295,6 +295,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + test @@ -307,6 +311,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -322,6 +330,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -338,6 +350,10 @@ ${apache-directory-server.version} test + + commons-lang + commons-lang + dom4j dom4j diff --git a/shims/0.23/pom.xml b/shims/0.23/pom.xml index 7ef46703f162..433263b05ecb 100644 --- a/shims/0.23/pom.xml +++ b/shims/0.23/pom.xml @@ -179,6 +179,10 @@ com.sun.jersey jersey-client + + commons-lang + commons-lang + @@ -190,6 +194,10 @@ org.slf4j slf4j-log4j12 + + commons-lang + commons-lang + commons-logging commons-logging @@ -199,6 +207,12 @@ org.apache.tez tez-mapreduce + + + commons-lang + commons-lang + + true @@ -248,6 +262,10 @@ commons-beanutils commons-beanutils + + commons-lang + commons-lang + org.slf4j slf4j-log4j12 diff --git a/standalone-metastore/metastore-common/pom.xml b/standalone-metastore/metastore-common/pom.xml index 07cbd5a69dcd..2ff92c231287 100644 --- a/standalone-metastore/metastore-common/pom.xml +++ b/standalone-metastore/metastore-common/pom.xml @@ -564,29 +564,6 @@ - - org.apache.maven.plugins - maven-enforcer-plugin - - - enforce-banned-dependencies - - enforce - - - - - - - com.google.code.findbugs:annotations - - - - true - - - - org.apache.maven.plugins maven-jar-plugin diff --git a/standalone-metastore/metastore-server/pom.xml b/standalone-metastore/metastore-server/pom.xml index 08cff9453ab5..7bc9c834c1d0 100644 --- a/standalone-metastore/metastore-server/pom.xml +++ b/standalone-metastore/metastore-server/pom.xml @@ -383,6 +383,10 @@ bouncycastle bcprov-jdk15 + + commons-lang + commons-lang + @@ -656,29 +660,6 @@ - - org.apache.maven.plugins - maven-enforcer-plugin - - - enforce-banned-dependencies - - enforce - - - - - - - com.google.code.findbugs:annotations - - - - true - - - - org.apache.maven.plugins maven-surefire-plugin diff --git a/standalone-metastore/metastore-tools/metastore-benchmarks/pom.xml b/standalone-metastore/metastore-tools/metastore-benchmarks/pom.xml index 5e7711fa84da..5da8df96a803 100644 --- a/standalone-metastore/metastore-tools/metastore-benchmarks/pom.xml +++ b/standalone-metastore/metastore-tools/metastore-benchmarks/pom.xml @@ -103,6 +103,10 @@ maven-jxr-plugin 2.5 + + commons-lang + commons-lang + org.apache.velocity velocity-tools diff --git a/standalone-metastore/pom.xml b/standalone-metastore/pom.xml index e23f779d0c1b..0b5106f8092c 100644 --- a/standalone-metastore/pom.xml +++ b/standalone-metastore/pom.xml @@ -583,6 +583,36 @@ maven-checkstyle-plugin ${maven.checkstyle.plugin.version} + + org.apache.maven.plugins + maven-enforcer-plugin + + + enforce-banned-dependencies + + enforce + + + + + + + com.google.code.findbugs:annotations + + + + + commons-lang:commons-lang + + true + Banned commons-lang:commons-lang dependency/transitive dependency was found! + + + true + + + + diff --git a/streaming/pom.xml b/streaming/pom.xml index 63ff4e8cbe69..2a083d3cc61e 100644 --- a/streaming/pom.xml +++ b/streaming/pom.xml @@ -150,11 +150,23 @@ org.apache.tez tez-mapreduce test + + + commons-lang + commons-lang + + org.apache.tez tez-dag test + + + commons-lang + commons-lang + +