This is an automated email from the ASF dual-hosted git repository. stoty pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/phoenix-connectors.git
The following commit(s) were added to refs/heads/master by this push: new 4da44c7 PHOENIX-6831 Remove bogus Ant dependency from hive connector test util 4da44c7 is described below commit 4da44c78181f2b76c525b1694f962fa909d2d596 Author: Istvan Toth <st...@apache.org> AuthorDate: Wed Nov 9 08:50:03 2022 +0100 PHOENIX-6831 Remove bogus Ant dependency from hive connector test util --- phoenix-hive-base/pom.xml | 7 ------- .../src/it/java/org/apache/hadoop/hive/ql/QTestUtil.java | 3 +-- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/phoenix-hive-base/pom.xml b/phoenix-hive-base/pom.xml index 5521583..8903e7e 100644 --- a/phoenix-hive-base/pom.xml +++ b/phoenix-hive-base/pom.xml @@ -181,13 +181,6 @@ </exclusion> </exclusions> </dependency> - <!-- Bogus test dependencies from the copied Hive QTestUtil --> - <dependency> - <groupId>org.apache.ant</groupId> - <artifactId>ant</artifactId> - <scope>provided</scope> - <version>1.9.1</version> - </dependency> <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-api</artifactId> diff --git a/phoenix-hive-base/src/it/java/org/apache/hadoop/hive/ql/QTestUtil.java b/phoenix-hive-base/src/it/java/org/apache/hadoop/hive/ql/QTestUtil.java index 54d16de..23fec0b 100644 --- a/phoenix-hive-base/src/it/java/org/apache/hadoop/hive/ql/QTestUtil.java +++ b/phoenix-hive-base/src/it/java/org/apache/hadoop/hive/ql/QTestUtil.java @@ -121,7 +121,6 @@ import org.apache.logging.log4j.util.Strings; import org.apache.phoenix.compat.CompatUtil; import org.apache.phoenix.compat.HiveCompatUtil; import org.apache.phoenix.compat.MyResult; -import org.apache.tools.ant.BuildException; import org.apache.zookeeper.WatchedEvent; import org.apache.zookeeper.Watcher; import org.apache.zookeeper.ZooKeeper; @@ -2176,7 +2175,7 @@ public class QTestUtil { } int pos = Arrays.binarySearch(cachedQvFileList, tname, String.CASE_INSENSITIVE_ORDER); if (pos >= 0) { - throw new BuildException("Unexpected file list element: " + cachedQvFileList[pos]); + throw new IllegalArgumentException("Unexpected file list element: " + cachedQvFileList[pos]); } List<String> result = null; for (pos = (-pos - 1); pos < cachedQvFileList.length; ++pos) {