kishoreg edited a comment on issue #4159: Add multiple locks which can be obtained based on segment name in lookupOrCreateFSM URL: https://github.com/apache/incubator-pinot/pull/4159#issuecomment-486420119 My bad, I thought there were two if blocks (outer and inner). In other words, we dont need a lock to check the existence in fsmMap we need the lock if the fsm does not exists. But it's not a big deal in this code path.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
