Praveenkumar76 commented on PR #25935: URL: https://github.com/apache/pulsar/pull/25935#issuecomment-4642354554
Hi @lhotari, I've completed the initial research! I dug through the codebase to map out exactly how time is being used, identified which specific areas are hurting testability (like cache TTLs and connection timeouts), and put together the architectural proposal we discussed. You can read the full breakdown here: [check this out ](https://gist.github.com/Praveenkumar76/bafd70c30710cf26951e8022f12929b8) Let me know if you agree with the categorization between the 'Diagnostics' bucket and the 'Business Logic' bucket, and if the constructor injection approach sounds like the right path forward for future PRs! -- 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]
