[ 
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)

Reply via email to