hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anoop Sam John (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-18825) Use HStoreFile instead of StoreFile in our own code base and remove unnecessary methods in StoreFile interface
Date Fri, 22 Sep 2017 05:07:00 GMT

    [ https://issues.apache.org/jira/browse/HBASE-18825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16175924#comment-16175924
] 

Anoop Sam John commented on HBASE-18825:
----------------------------------------

bq.What about StoreFile? I'm thinking about alternate HFile implementations. What if someone
wants to plug in a columnar-based file format? Or we want to do a V4 of HFile. This is harder
for me to swallow.
When we need new format or a V4, its the HFileReader and HFileWriter what will have to change.
 HFile on top of it gives top level APIs for creation of readers writer etc.  StoreFile is
more on top which is having APIs for traversing through next next Cells etc.  So I dont think
that we use our impl rather than interface will break any possibility for new HFile version..
  Here we have interface so as to pass to CPs.  So am +1

> Use HStoreFile instead of StoreFile in our own code base and remove unnecessary methods
in StoreFile interface
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-18825
>                 URL: https://issues.apache.org/jira/browse/HBASE-18825
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>             Fix For: 2.0.0-alpha-4
>
>         Attachments: HBASE-18825.patch, HBASE-18825-v1.patch, HBASE-18825-v2.patch, HBASE-18825-v3.patch,
HBASE-18825-v3.patch
>
>
> Use generic types to avoid too many casts.



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

Mime
View raw message