You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Hadoop QA (JIRA)" <ji...@apache.org> on 2015/05/02 06:34:12 UTC

[jira] [Commented] (HADOOP-9966) Refactor XDR code into XDRReader and XDRWriter

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

Hadoop QA commented on HADOOP-9966:
-----------------------------------

\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:red}-1{color} | patch |   0m  0s | The patch command could not apply the patch during dryrun. |
\\
\\
|| Subsystem || Report/Notes ||
| Patch URL | http://issues.apache.org/jira/secure/attachment/12603911/HADOOP-9966.002.patch |
| Optional Tests | javadoc javac unit findbugs checkstyle |
| git revision | trunk / f1a152c |
| Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/6311/console |


This message was automatically generated.

> Refactor XDR code into XDRReader and XDRWriter
> ----------------------------------------------
>
>                 Key: HADOOP-9966
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9966
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: nfs
>            Reporter: Haohui Mai
>            Assignee: Haohui Mai
>         Attachments: HADOOP-9966.000.patch, HADOOP-9966.001.patch, HADOOP-9966.002.patch
>
>
> Several methods in the current XDR class have ambiguous semantics. For example, Size() returns the actual size of internal byte array. The actual size of current buffer, is also affected by read requests, which pull data out of the buffer.
> These ambiguous semantics makes removing redundant copies on the NFS paths difficult.
> This JIRA proposes to decompose the responsibilities of XDR into two separate class: XDRReader and XDRWriter. The overall designs should closely follow Java's *Reader / *Writer classes.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)