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

HBase Review Board commented on HBASE-3211:
-------------------------------------------

Message from: "Jonathan Gray" <jg...@apache.org>


bq.  On 2010-11-10 16:01:22, Nicolas wrote:
bq.  > trunk/src/main/java/org/apache/hadoop/hbase/KeyValue.java, line 1199
bq.  > <http://review.cloudera.org/r/1208/diff/1/?file=17147#file17147line1199>
bq.  >
bq.  >     Would it be more straightforward to have a ReturnCode.MODIFY, that 
signals an include but suggests that the program must call modifyKeyValue() to 
get the transformed data.  Maybe this is too much of a one-off case...
bq.  
bq.  Jonathan Gray wrote:
bq.      Not sure I completely follow.  You're saying the modification would 
happen outside the filter?  No one needs to call modifyKeyValue() to get the 
transformed data, it's done in the filter.
bq.      
bq.      In any case, yeah, I would not be for adding another ReturnCode just 
for this.
bq.  
bq.  Nicolas wrote:
bq.      I suggested this alternative because users normally expect filters to 
do immutable operations on the data itself, and you're introducing side 
effects.  If we stay with this paradigm, it's probably best to add a note in 
Filter.filterKeyValue() that the KeyValue may be modified.

But a user would have to knowingly use this filter, right?  And the filter only 
has one purpose of mutating the KVs.  I do agree with what you're saying at 
some level but not sure what a note in the interface would do.  This is so if 
writing other filters, you would know that other filters in the chain could 
modify the KV?  How would you behave differently then in that case?


- Jonathan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1208/#review1896
-----------------------------------------------------------





> Key (Index) Only Fetches
> ------------------------
>
>                 Key: HBASE-3211
>                 URL: https://issues.apache.org/jira/browse/HBASE-3211
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Kannan Muthukkaruppan
>            Assignee: Jonathan Gray
>             Fix For: 0.90.0
>
>         Attachments: HBASE-3211-v2.patch, HBASE-3211-v3.patch
>
>
> When you retrieve data from HBase you get Key (Row+Column+Timestamp) + 
> Values. 
> It would be nice to have a mode where we only fetch the keys (i.e. the index) 
> but not the values.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to