vvysotskyi commented on a change in pull request #2216:
URL: https://github.com/apache/drill/pull/2216#discussion_r626021016
##########
File path: contrib/storage-kudu/pom.xml
##########
@@ -57,7 +57,7 @@
<dependency>
<groupId>org.apache.kudu</groupId>
<artifactId>kudu-client</artifactId>
- <version>1.3.0</version>
+ <version>1.14.0</version>
Review comment:
Could you please confirm that with the update Drill is able to connect
to Kudu? Also, there are no enabled unit tests for Kudu, existing ones are
disabled and require running Kudu server.
Could you please update tests to start Kudu using testcontainers, so we will
be sure that it is still working correctly?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
