[
https://issues.apache.org/jira/browse/HADOOP-5721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12703235#action_12703235
]
Konstantin Shvachko commented on HADOOP-5721:
---------------------------------------------
# In FSEditLog methods open(), addNewEditLogStream(), createEditLogFile()
should not be public. Since you are changing them could you please also make
them package private.
# Long lines in new methods, should be 80 chars.
# Indentation should be 2, not 4 like in addNewEditLogStream(),
createEditLogFile() and EditLogFileOutputStream(File name).
> Provide EditLogFileInputStream and EditLogFileOutputStream as independent
> classes
> ----------------------------------------------------------------------------------
>
> Key: HADOOP-5721
> URL: https://issues.apache.org/jira/browse/HADOOP-5721
> Project: Hadoop Core
> Issue Type: Improvement
> Components: dfs
> Affects Versions: 0.19.1
> Reporter: Luca Telloli
> Priority: Minor
> Fix For: 0.21.0
>
> Attachments: HADOOP-5721.patch, HADOOP-5721.patch, HADOOP-5721.patch,
> HADOOP-5721.patch, HADOOP-5721.patch
>
>
> EditLogFileInputStream and EditLogFileOutputStream are currently part of
> FSEditLog. With this patch we want to extract them and provide as independent
> classes.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
