GitHub user bjmb opened a pull request:
https://github.com/apache/tez/pull/22
[WIP][TEZ-3904] An API to update tokens for Tez AM and the DAG
This PR has ended up bigger than what I expected so I wanted to ask for
some feedback before going forward. This is what is does mainly:
* Adds `updateAMCredentials` and `updateDAGCredentials` to `TezClient`.
This is not the final API but this functions will be useful anyway to send the
credentials
* Add two corresponding functions to the DAGClientAMProtocol
* Add a `SystemEventHandler` in the runtime internals to process the
`UpdateCredentialsEvent`s
* Add the `UpdateCredentialsEvent` event and some transitions to the
`TaskImpl`'s state machine that represent the credentials being updated.
TODO:
* Add more tests. Will do after the feedback
* Log to history the credentials change?
* I've added some logic to renew the session credentials, these credentials
should be updated as well?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bjmb/tez TEZ-3904
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tez/pull/22.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #22
----
commit 6b50f27fda0c1a8698f0aed0f5e63f5f8413ba78
Author: Jaume Marhuenda <jaumemarhuenda@...>
Date: 2018-06-08T18:12:44Z
[TEZ-3904] An API to update tokens for Tez AM and the DAG
----
---
