hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tom White (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-938) too many SequenceFile.createWriter() methods
Date Wed, 30 Jan 2008 16:18:34 GMT

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

Tom White commented on HADOOP-938:

bq. Another approach would be to add an open() method, and use SequenceFile.Writer setter

I like this best, since it is simple. I think adding an open() method is good since it also
makes Writer (and Reader) easier to subclass (HADOOP-2739). Currently they do IO in the constructor
which can make it impossible for a subclass to do its initialization first - see https://issues.apache.org/jira/browse/HADOOP-1398?focusedCommentId=12559089#action_12559089.

> too many SequenceFile.createWriter() methods
> --------------------------------------------
>                 Key: HADOOP-938
>                 URL: https://issues.apache.org/jira/browse/HADOOP-938
>             Project: Hadoop Core
>          Issue Type: Improvement
>          Components: io
>            Reporter: Doug Cutting
> There are too many SequenceFile.createWriter() method signatures.  This method has two
required paramters: a Configuration and a Path.  It has one obsolete parameter: a FileSystem.
 And it has five optional parameters: CompressionType, CompressionCodec, Progress, replication,
and metadata.
> We should remove the obsolete parameter and make all optional parameters into setters.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message