kevinrr888 commented on PR #5171: URL: https://github.com/apache/accumulo/pull/5171#issuecomment-2541853708
> LGTM, I'm not sure how doable it is but looking at this I'm wondering if it would be possible to write a test that verifies the correct number of transaction runners are started if the pool is resized. That could always be done as a follow on or maybe just added as part of #5130 as that will be refactoring all the thread pool stuff anyways Yeah, I considered that as well. I think it would be a doable test. I think most likely I'll be adding tests for all the changes for #5130 anyways so would probably be easiest to include with those changes (would have the downside of only testing in 4.0+ though). -- 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]
