joshelser commented on a change in pull request #30:
URL: https://github.com/apache/phoenix-queryserver/pull/30#discussion_r416089071



##########
File path: 
queryserver-it/src/it/java/org/apache/phoenix/end2end/SecureQueryServerPhoenixDBIT.java
##########
@@ -324,6 +324,18 @@ public static synchronized void stopKdc() throws Exception 
{
 
     @Test
     public void testBasicReadWrite() throws Exception {
+        File file = new File(".");
+        runShellScript("python", Paths.get(file.getAbsolutePath(), "src", 
"it", "bin", "test_phoenixdb.py").toString());

Review comment:
       If it's easier, we could also bind some action to `pre-integration-test` 
and `post-integration-test` which start a PQS instance. I remember trying to 
debug failing python tests were a major pain in the rear with the I/O 
redirection flowing back through the Java maven process.




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


Reply via email to