jai1 commented on issue #1821: Closing client when Proxyconnection is disconnected URL: https://github.com/apache/incubator-pulsar/pull/1821#issuecomment-394028480 I understand the resource aspect of it and agree that instead of having one client we can extract the connectionPool part out of the client and use one connection pool per proxy connection But we still need one connection pool per proxy connection since the broker authenticates the client and the proxy if `forwardAuthorizationCredentials` is set to true and then the connection remains open throughout the lifetime of the proxy connection. So if Client A and Client B connect to Proxy we need one connection pool for A (since its A's credentials are authorized during handleConnect) and one connection pool for B. If we are in agreement let me know and I will start the work of having one connection pool instead of one client per proxy connection.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
