[
https://issues.apache.org/jira/browse/HBASE-15069?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jonathan Hsieh updated HBASE-15069:
-----------------------------------
Description:
There are a plethora of different static methods sprinkled through out
HStoreFile and HFile, and many tests that have extraneous calls to 'new
CacheConfig(conf)' or essentially extraneous FileSystem arguments threaded
through out the code.
This patch forces all creation to go through, limits the exposure of HFile
Reader and Writer Builders, and eliminates all static Builder constructors. It
also forces all HFile reader/writer uses outside of the o.a.h.h.io.hfile
package to use the StoreFile Readers
was:
There are a plethora of different static methods sprinkled through out
HStoreFile and HFile, and many tests that have extraneous calls to 'new
CacheConfig(conf)' or essentially extraneous FileSystem arguments threaded
through out the code.
This patch forces all creation to go through, limits the exposure of HFile
Reader and Writer Builders, and eliminates all static Builder constructors.
> Unify HFile Writer and Reader creation patterns
> -----------------------------------------------
>
> Key: HBASE-15069
> URL: https://issues.apache.org/jira/browse/HBASE-15069
> Project: HBase
> Issue Type: Bug
> Components: HFile
> Affects Versions: 2.0.0
> Reporter: Jonathan Hsieh
> Assignee: Jonathan Hsieh
> Fix For: 2.0.0
>
> Attachments: hbase-15069.patch
>
>
> There are a plethora of different static methods sprinkled through out
> HStoreFile and HFile, and many tests that have extraneous calls to 'new
> CacheConfig(conf)' or essentially extraneous FileSystem arguments threaded
> through out the code.
> This patch forces all creation to go through, limits the exposure of HFile
> Reader and Writer Builders, and eliminates all static Builder constructors.
> It also forces all HFile reader/writer uses outside of the o.a.h.h.io.hfile
> package to use the StoreFile Readers
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
