JinwooHwang commented on PR #7960: URL: https://github.com/apache/geode/pull/7960#issuecomment-3567842610
@leonfin . Excellent fix for a critical edge case! This properly addresses the NullPointerException in partitioned region queries where afterIndexLookup() can be called without a corresponding beforeIndexLookup() due to distributed execution across threads/nodes. The fix follows the right principle: diagnostic/observability code (index tracking) should never cause functional failures. By gracefully handling the uninitialized ThreadLocal, queries now succeed even when index tracking data is unavailable. The added test cases clearly demonstrate both the failure scenario and the fix. Well-documented inline comments explain exactly when this can occur in production. I appreciate your great work ensuring query reliability! -- 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]
