Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/942#discussion_r139273842
--- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java
---
@@ -100,6 +101,14 @@ public void run() {
return dir.getAbsolutePath() + File.separator + dirName;
}
+ /**
+ * Sets zookeeper server and client SASL test config properties.
+ */
+ public static void setZookeeperSaslTestConfigProps() {
--- End diff --
Maybe it's possible to create separate test zk util class with this method
and also setup for jaas property (so jaas config is not repeated twice in the
code) and keep it in the same package where we test zk?
---
