[
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 HFile Reader and Writer Builders,
eliminates all static Builder constructors, and limits the exposure
Reader/Writers . 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. It
also forces all HFile reader/writer uses outside of the o.a.h.h.io.hfile
package to use the StoreFile Readers
> 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 HFile Reader and Writer
> Builders, eliminates all static Builder constructors, and limits the exposure
> Reader/Writers . 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)
