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

Ted Yu commented on HBASE-7899:
-------------------------------

{code}
+   * backing bytes copied from the steam.
{code}
Typo: steam

For CellTool:
{code}
+  public static CellScanner createCellScanner(final Iterator<Cell> cells) {
...
+      public boolean next() {
+        return this.iterator.hasNext();
+      }
{code}
Is the implementation for next() correct ?
{code}
+      public Cell getDeepCopy() {
+        // TODO: Fix to do a deep copy.  Does the CellIterable passed have to 
be against deep copies?
+        return get();
+      }
{code}
Do you plan to address the above in a follow-on issue ?
                
> Cell block building tools: Cell codec and means of iterating an objects Cells
> -----------------------------------------------------------------------------
>
>                 Key: HBASE-7899
>                 URL: https://issues.apache.org/jira/browse/HBASE-7899
>             Project: HBase
>          Issue Type: Task
>          Components: IPC/RPC
>            Reporter: stack
>            Assignee: stack
>            Priority: Blocker
>             Fix For: 0.96.0
>
>         Attachments: 7899.txt
>
>
> In our rpc specification, 
> https://docs.google.com/document/d/1-1RJMLXzYldmHgKP7M7ynK6euRpucD03fZ603DlZfGI/edit#,
>  we talk of passing lots of Cells outside of protobuf.  We do it using Cell 
> blocks. There are blobs filled with Cells that are encoded variously and 
> optionally compressed.
> This issue is about adding to hbase-common basic facility for building Cell 
> blocks.  It includes a codec Interface for encoding/decoding Cell blocks as 
> well as a means of iterating an objects Cell content (prefix-tree's 
> CellScanner).
> This is a subtask of HBASE-7898 Serializing Cells over RPC

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to