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

jirapos...@reviews.apache.org commented on HBASE-5358:
------------------------------------------------------



bq.  On 2012-02-09 07:02:48, Michael Stack wrote:
bq.  > src/main/java/org/apache/hadoop/hbase/io/HbaseObjectWritable.java, line 
446
bq.  > <https://reviews.apache.org/r/3811/diff/2/?file=73392#file73392line446>
bq.  >
bq.  >     We are changing what is on the line now when we serialize?  All 
arrays now serialize their type ahead of array length?  In our public API what 
arrays will this effect?  Do you know?  Does it require our upping the version 
on the Interface that uses HBaseObjectWritable?

We are not changing the serialized bytes for the arrays that are already in 
CLASS_TO_CODE. This only affects the arrays that are not defined there. For 
them, when you serialize, the deserializer throws an erroneous 
ClassNotFoundException. So, I don't think wire compatibility will be affected. 


- enis


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3811/#review4960
-----------------------------------------------------------


On 2012-02-09 01:38:18, enis wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/3811/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-02-09 01:38:18)
bq.  
bq.  
bq.  Review request for hbase.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  HBaseObjectWritable can encode Writable[]'s but, but cannot encode A[] 
where A extends Writable. This becomes an issue for example when adding a 
coprocessor method which takes A[] (see HBASE-5352).
bq.  
bq.  
bq.  This addresses bug HBASE-5358.
bq.      https://issues.apache.org/jira/browse/HBASE-5358
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    src/test/java/org/apache/hadoop/hbase/io/TestHbaseObjectWritable.java 
78513ce 
bq.    src/main/java/org/apache/hadoop/hbase/io/HbaseObjectWritable.java 
260f982 
bq.  
bq.  Diff: https://reviews.apache.org/r/3811/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  enis
bq.  
bq.


                
> HBaseObjectWritable should be able to serialize generic arrays not defined 
> previously
> -------------------------------------------------------------------------------------
>
>                 Key: HBASE-5358
>                 URL: https://issues.apache.org/jira/browse/HBASE-5358
>             Project: HBase
>          Issue Type: Improvement
>          Components: coprocessors, io
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>
> HBaseObjectWritable can encode Writable[]'s but, but cannot encode A[] where 
> A extends Writable. This becomes an issue for example when adding a 
> coprocessor method which takes A[] (see HBASE-5352). 

--
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

        

Reply via email to