Skip to content

Commit d49377e

Browse files
cxzl25turboFei
authored andcommitted
[KYUUBI #2772] Kyuubi Spark TPC-H Connector - use log4j1
### _Why are the changes needed?_ `kyuubi-spark-connector-tpch` uses log4j1 configuration in test, but does not introduce log4j1 dependency, which leads to some Spark INFO output when executing UT. ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [x] Add screenshots for manual tests if appropriate - [x] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request Closes #2772 from cxzl25/tpch_bind_log4j1. Closes #2772 367f657 [sychen] tpch use log4j1 Authored-by: sychen <sychen@ctrip.com> Signed-off-by: Fei Wang <fwang12@ebay.com>
1 parent b817fcf commit d49377e

File tree

1 file changed

+7
-0
lines changed
  • extensions/spark/kyuubi-spark-connector-tpch

1 file changed

+7
-0
lines changed

extensions/spark/kyuubi-spark-connector-tpch/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,13 @@
6161
<artifactId>guava</artifactId>
6262
</dependency>
6363

64+
<dependency>
65+
<groupId>org.apache.spark</groupId>
66+
<artifactId>spark-core_${scala.binary.version}</artifactId>
67+
<type>test-jar</type>
68+
<scope>test</scope>
69+
</dependency>
70+
6471
<dependency>
6572
<groupId>org.apache.spark</groupId>
6673
<artifactId>spark-catalyst_${scala.binary.version}</artifactId>

0 commit comments

Comments
 (0)