jameshartig commented on PR #1755: URL: https://github.com/apache/cassandra-gocql-driver/pull/1755#issuecomment-2941497973
> In this PR I kept the old logger printf based interface as deprecated and my reasoning was that this is something that is probably widely used and breaking it could mean users feel discouraged to upgrade to 2.0. I see what you mean but I'm concerned that if we switch logging and keep them in there then we're just continuing the dependence. I don't really see a great solution since we want to keep the existing logs unless we want to add 2 log calls to every spot where we log. Having the log message as `"connection pool filling failed %s (%s): %v"` in the new structured logs is less than ideal. Maybe we can just make it very clear that the messages will change and no one should rely on the messages being static? -- 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]
