[
https://issues.apache.org/jira/browse/TEZ-3025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15088513#comment-15088513
]
Bikas Saha commented on TEZ-3025:
---------------------------------
Unnecessary diff?
{code}- public void
runInputInitializers(List<RootInputLeafOutput<InputDescriptor,
InputInitializerDescriptor>>
+ public void runInputInitializers(List<RootInputLeafOutput<InputDescriptor,
InputInitializerDescriptor>>
inputs) throws TezException {
{code}
Ideally, we could have had a separate method for initializer initialization
instead of having to do it during object creation. That clarifies the contract
explicitly. But that would be incompatible. +1 for the fix.
> InputInitializer creation should use the dag ugi
> ------------------------------------------------
>
> Key: TEZ-3025
> URL: https://issues.apache.org/jira/browse/TEZ-3025
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Siddharth Seth
> Assignee: Siddharth Seth
> Attachments: TEZ-3025.1.txt, TEZ-3025.2.txt
>
>
> Currently, running the initialize methods of InputInitializers happens within
> the dag ugi. The creation of the initializer needs to do the same.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)