[GitHub] [storm] kishorvpatil opened a new pull request #3301: MINOR: Remove unwanted creds update check at worker start

[GitBox <gi...@apache.org>]

kishorvpatil opened a new pull request #3301:
URL: https://github.com/apache/storm/pull/3301


   ## What is the purpose of the change
   
   The callback method to request credentials in the _loadWorker_ method is not required as it receives _initialCredentials_ as parameter. and within _TASK_CREDENTIALS_POLL_SECS_ schedule recurring call to _checkCredentialsChanged_ is setup. Also, all executors at init time receive initial credentials so no need for this additional call. 
   
   ## How was the change tested
   
   Manually launched topology to verify if _setCredentials_ is called on all executors correctly on the start of workers.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [storm] Ethanlm merged pull request #3301: MINOR: Remove unwanted creds update check at worker start

[GitBox <gi...@apache.org>]

Ethanlm merged pull request #3301:
URL: https://github.com/apache/storm/pull/3301


   


----------------------------------------------------------------
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:
users@infra.apache.org