hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lars Hofhansl (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-5670) Have Mutation implement the Row interface.
Date Thu, 29 Mar 2012 05:43:53 GMT

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

Lars Hofhansl commented on HBASE-5670:
--------------------------------------

Thanks Stack.
Since you can pass List<Row> to a method that takes List<? extends Row> it'll
work without the need to recompile anything.
The client<->server communication is done via HConnection.processBatch, which is public
and actually already takes List<? extends Row>. Seems the List<Row> interface
in HTable was just an oversight.

                
> Have Mutation implement the Row interface.
> ------------------------------------------
>
>                 Key: HBASE-5670
>                 URL: https://issues.apache.org/jira/browse/HBASE-5670
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Lars Hofhansl
>            Assignee: Lars Hofhansl
>            Priority: Trivial
>             Fix For: 0.94.1
>
>         Attachments: 5670-0.94.txt, 5670-trunk.txt
>
>
> In HBASE-4347 I factored some code from Put/Delete/Append in Mutation.
> In a discussion with a co-worker I noticed that Put/Delete/Append still implement the
Row interface, but Mutation does not.
> In a trivial change I would like to move that interface up to Mutation, along with changing
HTable.batch(List<Row>) to HTable.batch(List<? extends Row>) (HConnection.processBatch
takes List<? extends Row> already anyway), so that HTable.batch can be used with a list
of Mutations.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message