stoty commented on pull request #112:
URL: https://github.com/apache/phoenix-tephra/pull/112#issuecomment-822144285


   1. There seems to be a pom error that causes the test to fail.
   
   2. Did you have to make changes in the code for 2.4 compared to 2.3 ? If 
not, then it'd be better to turn the 2.3 module into 2.3-base, and use that as 
a base for the actual 2.3 and 2.4 modules to avoid code duplication. (see the 
2.0-base module for an example)


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


Reply via email to