shahrs87 commented on PR #1610: URL: https://github.com/apache/phoenix/pull/1610#issuecomment-1602901308
In bunch of tests I have replaced `PHOENIX_JDBC_URL` with `getUrl()` method. `PHOENIX_JDBC_URL` is evaluated to `jdbc:phoenix:localhost;test=true` which doesn't have zookeeper port number in the URL. It defaults to 2181 which is the default port number. Before the patch in PhoenixTestDriver, we create CQSI object once, cache it in PhoenixTestDriver and all the other calls uses the same CQSI object. But after the patch, we maintain a cache of `ConnectionInfo -> ConnectionQueryServices` within PhoenixTestDriver. To construct the right ConnectionInfo we need the actual zookeeper port number. Thats why we can't use `PHOENIX_JDBC_URL` anymore. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
