JinwooHwang commented on PR #7957: URL: https://github.com/apache/geode/pull/7957#issuecomment-3607725872
Thank you so much @sboorlagadda for the thorough review and approval! You're absolutely right about the null check - I've addressed both of your suggestions in the latest commit: 1. Added null check for `platformOsBean` before calling `getAvailableProcessors()` (consistent with the pattern for `unixOsBean`) 2. Enhanced the error message to clarify that initialization failure affects statistics collection but not core Geode functionality Really appreciate your attention to detail and helping make this code more robust! -- 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]
