[PHOENIX-5826] Remove guava from queryserver

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: queryserver-6.0.0
• Fix Version/s: queryserver-6.0.0
• Component/s: queryserver
• Assignee: Istvan Toth
• Reporter: Istvan Toth
• Created: 08/Apr/20 13:30
• Updated: 03/Aug/21 08:35
• Resolved: 19/May/20 06:56

Description

The queryserver repo has to work with multiple phoenix branches.
HBase and Hadoop master has already migrated to guava27/28.

Queryserver has to work both with the 4.x branch, which will probably have to stay on the old guava version, and master, which will have to be upgraded to 28.

The queryserver codebase does not work with Guava 28.

The simplest way to solve this problem is to remove guava from the queryserver modules.
Most of the guava uses are trivial to replace , but unfortunately changing the host information that load-balancer stores in zookeeper is a non backward compatible change . The HostAndPort object would have to be replaced by something like InetSocketAddress .

Since we are before the first release of the unbundled queryserver, I think that this a great time to get this done with minimal disruption to the users.

Issue Links

• relates to: PHOENIX-5965 Remove Guava from Phoenix-Connectors
• links to: GitHub Pull Request #33