absurdfarce commented on PR #2037: URL: https://github.com/apache/cassandra-java-driver/pull/2037#issuecomment-3276447370
After discussing the PR containing my proposed changes (referenced [above](https://github.com/apache/cassandra-java-driver/pull/2037#issuecomment-3264545632)) with @SiyaoIsHiding there seemed to be broad agreement on the structural changes (essentially decoupling the request ID from the session name and treating it as a replacement for the CqlRequestHandler hash code only) but some disagreement on method parameters and a few naming questions. In order to avoid fragmenting the discussion I've merged those changes into this PR (see [this commit](https://github.com/apache/cassandra-java-driver/pull/2037/commits/0120f03f0a63872fdaf7964d02e5e32157929d88)). We'll continue the discussion on these remaining points here. I'll be closing the alternate PR with a similar note shortly. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
