[GitHub] [pinot] lucifer4j commented on pull request #11014: Fix double-checked locking in ConnectionFactory

["lucifer4j (via GitHub)" <gi...@apache.org>]

lucifer4j commented on PR #11014:
URL: https://github.com/apache/pinot/pull/11014#issuecomment-1626898927

   @Jackie-Jiang Hi! Sorry for the delay, I can add the link but I see the PR is already merged now.
   @ankitsultana I was working on this file as part of a different PR when I noticed this issue being pointed out by static analysis. AFAIU, this is a benign race condition as multiple PinotClientTransport's can co-exist without bugs.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org