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

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

Exactly Lars.   As a first step I am trying to remove the use from Compactor 
and HFileWriter.   (Why it is was needed in Compactor is because Writer accepts 
KV only).  I will attach patch for that to HBASE-11874.
Similar way will check each area one by one and try remove 
KVUtil#ensureKeyValue()
I think in 1.0 itself we should deprecate this. Even though KeyValueUtil is 
marker Private, Phoenix uses it. And we should get rid of the usage in Phoenix 
also.

> Avoid usage of KeyValueUtil#ensureKeyValue
> ------------------------------------------
>
>                 Key: HBASE-11871
>                 URL: https://issues.apache.org/jira/browse/HBASE-11871
>             Project: HBase
>          Issue Type: Improvement
>          Components: regionserver
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>
> Now we try to use diff impls of Cells (other than KV) in server side for 
> optimizations. (HBASE-11870  is one like this) Having code  
> KeyValueUtil#ensureKeyValue can be dangerous as it recreate the Cell and do 
> buffer copy. This is an umbrella issue.  Will raise small sub tasks in 
> specific areas and tackle this.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to