joshelser commented on issue #25: PHOENIX-5827 Add a maven repo to PQS
URL: 
https://github.com/apache/phoenix-queryserver/pull/25#issuecomment-611811455
 
 
   ad51a2b REALLY opened up the rabbit hole. Hadoop2/HBase1 have a big issue 
with Jetty6 leaking out of the classpath. We aren't going to fix that problem, 
so we either have to bandaid it or not support it in PQS.
   
   Since not supporting Phoenix 4 isn't an issue yet, we gotta work around it. 
I moved all ITs to their own module where Jetty6 can run wild, and anything 
which is touching our Jetty 9.4 classes into queryserver (so they can be 
shaded). AFAICT, this works, but is a pain in the butt.
   
   A `mvn verify` runs locally for me, but I still need to retest:
   * `mvn verify -Dphoenix.version=5.1.0-SNAPSHOT` still works
   * The assembly (and the new flag) still create reasonable tarballs

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


With regards,
Apache Git Services

Reply via email to