jenniferdai opened a new pull request #3770: Remove unnecessary init from crypter URL: https://github.com/apache/incubator-pinot/pull/3770 * Crypter is initialized in controller/server starter, so we do not need to init it here. The problem with initing here is that there can be a possible race condition where we are trying to decrypt another segment but the keys will have disappeared since we have now introduced concurrent segment downloads
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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]
