[
https://issues.apache.org/jira/browse/HIVE-20853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16677497#comment-16677497
]
Jaume M commented on HIVE-20853:
--------------------------------
I've tested this manually on a secure cluster. I can see the nodes getting
registered when they come up. I'm having some doubts about if this is the
correct solution:
*registerDag(applicationString, dagId) is what it's currently implemented and
needs an applicationString and a dagId to register the dag. However we only
have the dagId after the first task attempt, which maybe is too late but I
haven't seen evidence of this.
*registerCredentials(applicationString) if we implemented this API instead of
the previous one we wouldn't need the dagId and we could register the
crendentials independently of if the first task attempt has started. I think
this approach is slightly cleaner, wdyt [~sershe]?
> Expose ShuffleHandler.registerDag in the llap daemon API
> --------------------------------------------------------
>
> Key: HIVE-20853
> URL: https://issues.apache.org/jira/browse/HIVE-20853
> Project: Hive
> Issue Type: Improvement
> Components: llap
> Affects Versions: 3.1.0
> Reporter: Jaume M
> Assignee: Jaume M
> Priority: Critical
> Attachments: HIVE-20853.1.patch, HIVE-20853.2.patch,
> HIVE-20853.3.patch, HIVE-20853.4.patch
>
>
> Currently DAGs are only registered when a submitWork is called for that DAG.
> At this point the crendentials are added to the ShuffleHandler and it can
> start serving.
> However Tez might (and will) schedule tasks to fetch from the ShuffleHandler
> before anything of this happens and all this tasks will fail which may
> results in the query failing.
> This happens in the scenario in which a LlapDaemon just comes up and tez
> fetchers try to open a connection before a DAG has been registered.
> Adding this API will allow to register the DAG against the Daemon when the AM
> notices that a new Daemon is up.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
