virajjasani commented on a change in pull request #1103:
URL: https://github.com/apache/phoenix/pull/1103#discussion_r561745498



##########
File path: pom.xml
##########
@@ -1577,6 +1581,7 @@
         <hbase.version>${hbase-2.3.runtime.version}</hbase.version>
         <hbase.thirdparty.version>3.3.0</hbase.thirdparty.version>
         <zookeeper.version>3.5.7</zookeeper.version>
+        <hadoop.version>3.1.3</hadoop.version>

Review comment:
       > I do not have a firm opinon either way, if you think that we should 
simnply revert PHOENIX-6326, that is also possible.
   
   I also don't have firm opinion either. I think this might take a while doing 
testing and all, maybe bake in multibranch build for a couple of days and once 
we see things clear, maybe we should proceed. Given all the work you have done, 
i think good to proceed with updated Hadoop versions. Only if it turns out to 
be significant tests failing due to Hadoop upgrade, maybe we can revert and 
take some time with individual HBase profiles. If we go in this situation, 
please let me know, i will try to test specific HBase profile and try to help 
ensure the least latest Hadoop version that we can support.
   
   On the other hand, builds you are running currently themselves should be 
sufficient to know the stability I believe.




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