[jira] [Comment Edited] (HBASE-13900) duplicate methods between ProtobufMagic and ProtobufUtil

2015-06-19 Thread Andrew Purtell (JIRA)

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

Andrew Purtell edited comment on HBASE-13900 at 6/19/15 9:26 PM:
-

Not sure what's up with the javadoc warning either, I don't see it. Committed 
to master. Thanks for the patch [~gliptak]!


was (Author: apurtell):
Not sure what's up with the javadoc warning either, I don't see it. Committed 
to mater. Thanks for the patch [~gliptak]!

> duplicate methods between ProtobufMagic and ProtobufUtil
> 
>
> Key: HBASE-13900
> URL: https://issues.apache.org/jira/browse/HBASE-13900
> Project: HBase
>  Issue Type: Improvement
>Reporter: Gabor Liptak
>Assignee: Gabor Liptak
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-13900.1.patch
>
>
> Several methods duplicated between:
> hbase-protocol/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufMagic.java
> and
> hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
> isPBMagicPrefix()
> isPBMagicPrefix()
> lengthOfPBMagic()
> ProtobufUtil partically references ProtobufMagic, but it also has different 
> compare implementation ...
> Maybe this was based on packaging/signature need?



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


[jira] [Comment Edited] (HBASE-13900) duplicate methods between ProtobufMagic and ProtobufUtil

2015-06-22 Thread Andrew Purtell (JIRA)

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

Andrew Purtell edited comment on HBASE-13900 at 6/22/15 4:56 PM:
-

[~misty] did the necessary revert and recommit already. Thanks [~misty]!


was (Author: apurtell):
[~misty] did the necessary revert an recommit already. Thanks [~misty]!

> duplicate methods between ProtobufMagic and ProtobufUtil
> 
>
> Key: HBASE-13900
> URL: https://issues.apache.org/jira/browse/HBASE-13900
> Project: HBase
>  Issue Type: Improvement
>Reporter: Gabor Liptak
>Assignee: Gabor Liptak
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-13900.1.patch
>
>
> Several methods duplicated between:
> hbase-protocol/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufMagic.java
> and
> hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
> isPBMagicPrefix()
> isPBMagicPrefix()
> lengthOfPBMagic()
> ProtobufUtil partically references ProtobufMagic, but it also has different 
> compare implementation ...
> Maybe this was based on packaging/signature need?



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