captainzmc commented on a change in pull request #209: [LIVY-640] Add tests for 
ThriftServer
URL: https://github.com/apache/incubator-livy/pull/209#discussion_r319526712
 
 

 ##########
 File path: pom.xml
 ##########
 @@ -1009,6 +1007,20 @@
   </reporting>
 
   <profiles>
+    <profile>
 
 Review comment:
   Without the two profiles, both thriftserver and integration-test will be 
executed in ITs, which may cause timeouts.
   So we need these two profiles. Then separate thriftserver and 
integration-test in travis.yml. 
    Like this:
   Env: MVN_FLAG = '- Pthriftserver - DskipTests'
   Env: MVN_FLAG = '- Pintegration - test - Pcoverage - DskipTests'

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to