tez-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bjmb <...@git.apache.org>
Subject [GitHub] tez pull request #22: [WIP][TEZ-3904] An API to update tokens for Tez AM and...
Date Wed, 13 Jun 2018 23:59:14 GMT
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

----


---

Mime
View raw message