[ 
https://issues.apache.org/jira/browse/TEZ-3684?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jonathan Eagles updated TEZ-3684:
---------------------------------
    Attachment: TEZ-3684.1.patch

> Incorporate first pass non-essential TEZ-3334 pre-merge feedback
> ----------------------------------------------------------------
>
>                 Key: TEZ-3684
>                 URL: https://issues.apache.org/jira/browse/TEZ-3684
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-3684.1.patch
>
>
> Incorporates this feedback from TEZ-3334 
> https://issues.apache.org/jira/browse/TEZ-3334?focusedCommentId=15950550&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15950550
> - TezContainerLauncherImpl - ByteBuffer portInfo 
> =response.getAllServicesMetaData().get(conf.get(TezConfiguration.TEZ_AM_SHUFFLE_AUXILIARY_SERVICE_ID,TezConfiguration.TEZ_AM_SHUFFLE_AUXILIARY_SERVICE_ID_DEFAULT));
>  - Could be split into multiple calls, in case there's a possibility of a NPE.
> - TestShuffleHandlerJobs. Parameter name mrrTez - should really be tez*.
> - Variable renames. pendingInputsOfOnePartition is no longer a valid name.
> - InputHost.clearAndGetOnePartition - rename. This is no longer giving back a 
> single partition only.
> - pendingInputsOfOnePartition - name is likely not valid any longer



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to