[ 
https://issues.apache.org/jira/browse/TEZ-3813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16120590#comment-16120590
 ] 

Jonathan Eagles commented on TEZ-3813:
--------------------------------------

Couple of minor nits

Seems we can removed this commented out code
{code:title=FetchedInput.java}
+//  public long getActualSize() {
+//    return this.actualSize;
+//  }
+//
+//  public long getCompressedSize() {
+//    return this.compressedSize;
+//  }
{code}

We should add \@Override to this and others who override getSize
{code:title=MemoryFetchedInput.java}
public long getSize()
{code}

> Reduce Object size of MemoryFetchedInput for large jobs
> -------------------------------------------------------
>
>                 Key: TEZ-3813
>                 URL: https://issues.apache.org/jira/browse/TEZ-3813
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Muhammad Samir Khan
>            Assignee: Muhammad Samir Khan
>         Attachments: TEZ-3813.001.patch, TEZ-3813.002.patch, 
> TEZ-3813.003.patch, TEZ-3813.004.patch, TEZ-3813.005.patch
>
>
> Same as TEZ-3752 for the unordered case. MemoryFetchedInput has a 
> BoundedByteArrayOutputStream that is not used (only the underlying byte[] is 
> used).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to