timoninmaxim commented on pull request #9817: URL: https://github.com/apache/ignite/pull/9817#issuecomment-1042709471
Hi, this patch LGTM. But I'm not sure, but maybe we should add some smoke tests that check connect a client to multiple nodes to be confident that the timer isn't shared between channels. We configure heartbeat on ClientConfiguration level, where also partition awareness configured. Then it looks meaningful and maybe it can be useful for regression testing, WDYT? -- 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]
