stoty commented on pull request #45: URL: https://github.com/apache/phoenix-connectors/pull/45#issuecomment-753258480
I think that we are breaking much more than this already, even without this patch. - We have completely removed the connectors from Phoenix proper, and moved them to a new project, so both the old maven coordinates, and the installed JAR locations are gone anyways. - All but the hive connectors were included in the phoenix-client JAR, and we told users to use that. The new phoenix-client JAR doesn't include them for obvious reasons, and I currently have no plans reintroduce a new uber-uber-jar to connectors, due to the shading issues that the old one has. (see PHOENIX-5202 ) We will need update ALL connectors-related docs anyways. - Add a release note for core that points to the new -connectors (and -pqs repos) - Add/Update building docs for PQS and connectors, describing how the 4.x/master, Phoenix and HBase releases are handled. - Update the docs for each connector for the new repo and JARs / artifacts. - The Spark connectors docs just plain don't work as written, and possibly never did. - The Hive connector has serious conflicts with the HBase 2.0.0-alpha included in Hive 3.1.2 (and also with the ancient Hbase in 2.4 for Phoenix 4.x), which requires either hacking up the binary hive distro, or rebuilding it from source with an unmerged patch (see HIVE-24473), we'll also have to document that. However, I am not completely sure about the exact naming scheme in this PR, so if you'd prefer an alternative, I'd be happy to discuss that, even if keeping the old one is impossible. ---------------------------------------------------------------- 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]
