vdiravka commented on a change in pull request #2447:
URL: https://github.com/apache/drill/pull/2447#discussion_r797537220



##########
File path: contrib/storage-phoenix/pom.xml
##########
@@ -33,7 +33,7 @@
     <phoenix.version>5.1.2</phoenix.version>
     <!-- Keep the 2.4.2 to reduce dependency conflict -->
     <hbase.minicluster.version>2.4.2</hbase.minicluster.version>
-    <jetty.test.version>9.4.31.v20200723</jetty.test.version>
+    <jetty.test.version>9.4.44.v20210927</jetty.test.version>

Review comment:
       I've checked and `SecuredPhoenixTestSuite` works fine with this jetty 
version (dependencies in `drill-storage-phoenix` are still needed for that).
   
   So please remove `jetty.test.version` property and it's usage




-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to