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

Virajith Jalaparti updated HDFS-12591:
--------------------------------------
    Attachment: HDFS-12591-HDFS-9806.006.patch

Thanks for taking a look [~chris.douglas]. Posting a modified patch (v006) 
based on the comments -- it throws an {{IllegalArgumentException}} in 
{{LevelDBOptions.setConf}}, if {{DFS_PROVIDED_ALIASMAP_LEVELDB_PATH}} is not 
set correctly. Also, I removed the {{@VisibleForTesting}} for {{createDB}}, 
made it static and added a {{boolean}} parameter to determine if the db has to 
be created (when being written) or not (when being read).

> [READ] Implement LevelDBFileRegionFormat
> ----------------------------------------
>
>                 Key: HDFS-12591
>                 URL: https://issues.apache.org/jira/browse/HDFS-12591
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs
>            Reporter: Ewan Higgs
>            Assignee: Ewan Higgs
>            Priority: Minor
>         Attachments: HDFS-12591-HDFS-9806.001.patch, 
> HDFS-12591-HDFS-9806.002.patch, HDFS-12591-HDFS-9806.003.patch, 
> HDFS-12591-HDFS-9806.004.patch, HDFS-12591-HDFS-9806.005.patch, 
> HDFS-12591-HDFS-9806.006.patch
>
>
> The existing work for HDFS-9806 uses an implementation of the {{FileRegion}} 
> read from a csv file. This is good for testability and diagnostic purposes, 
> but it is not very efficient for larger systems.
> There should be a version that is similar to the {{TextFileRegionFormat}} 
> that instead uses LevelDB.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to