twdsilva commented on issue #567: PHOENIX-5134 Phoenix Connection Driver #normalize does not distinguis… URL: https://github.com/apache/phoenix/pull/567#issuecomment-522364296 I don't think we should create a new HConnection for a tenant specific connection, since its pretty heavy weight, and this could end up creating a lot of HConnections. Its fine for different tenant to use the sam CQSI as long as the rest of the config in the Properties is the same. If a client tries to connect to a cluster with a Properties object that is different than what is currently present in the cache, we should create a new CQSI instead of using the cached one that was created with a different configuration. Maybe you can use the Properties object to compute the hashCode of ConnectionInfo so that if a client passes in different Properties config a new HConnection is created?
---------------------------------------------------------------- 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] With regards, Apache Git Services
