storm-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [storm] kishorvpatil opened a new pull request #3301: MINOR: Remove unwanted creds update check at worker start
Date Wed, 01 Jul 2020 16:43:51 GMT

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



Mime
View raw message