stoty commented on pull request #964: URL: https://github.com/apache/phoenix/pull/964#issuecomment-739785063
> @stoty we should also bump `provided` dependencies of `phoenix-hbase-compat-x.y.z`'s hbase-client, hbase-common, hbase-server? > e.g bump hbase-server to 2.3.3 here https://github.com/apache/phoenix/blob/master/phoenix-hbase-compat-2.3.0/pom.xml#L70-L71 > Thought? No. Those are intentionally set to the lowest release of the HBase API variant that we are targeting- While updating it wouldn't break the code, it is not necessary, and would cause confusion. If there was an API change that affected us, we'd have to add a new a compatibility module instead. The provided HBase dependency versions here will get overridden with the proper versions from the phoenix parent module. ---------------------------------------------------------------- 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]
