[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-05-10 Thread Phabricator (JIRA)

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

Phabricator commented on HBASE-5736:


sc has closed the revision "HBASE-5736 [jira] 
ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly".

REVISION DETAIL
  https://reviews.facebook.net/D2649

To: tedyu, dhruba, JIRA, sc


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.94.0, 0.96.0
>
> Attachments: 5736-94.txt, HBASE-5736.D2649.1.patch, 
> HBASE-5736.D2649.2.patch, HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-13 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-0.94-security #9 (See 
[https://builds.apache.org/job/HBase-0.94-security/9/])
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use 
ByteBuffer correctly (Scott Chen) (Revision 1325067)

 Result = SUCCESS
tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.94.0, 0.96.0
>
> Attachments: 5736-94.txt, HBASE-5736.D2649.1.patch, 
> HBASE-5736.D2649.2.patch, HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-12 Thread Scott Chen (Commented) (JIRA)

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

Scott Chen commented on HBASE-5736:
---

Thanks for doing the 94 patch

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.94.0, 0.96.0
>
> Attachments: 5736-94.txt, HBASE-5736.D2649.1.patch, 
> HBASE-5736.D2649.2.patch, HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-11 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-0.94 #103 (See 
[https://builds.apache.org/job/HBase-0.94/103/])
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use 
ByteBuffer correctly (Scott Chen) (Revision 1325067)

 Result = SUCCESS
tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.94.0, 0.96.0
>
> Attachments: 5736-94.txt, HBASE-5736.D2649.1.patch, 
> HBASE-5736.D2649.2.patch, HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-0.94-security #8 (See 
[https://builds.apache.org/job/HBase-0.94-security/8/])
HBASE-5736 revert TestThriftServer doesn't compile (Revision 1310643)
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer 
correctly (Scott Chen) (Revision 1310638)

 Result = SUCCESS
tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java

tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-TRUNK-security #161 (See 
[https://builds.apache.org/job/HBase-TRUNK-security/161/])
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use 
ByteBuffer correctly (Scott Chen) (Revision 1310634)

 Result = FAILURE
tedyu : 
Files : 
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-0.94 #95 (See 
[https://builds.apache.org/job/HBase-0.94/95/])
HBASE-5736 revert TestThriftServer doesn't compile (Revision 1310643)
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer 
correctly (Scott Chen) (Revision 1310638)

 Result = FAILURE
tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java

tedyu : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Hudson (Commented) (JIRA)

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

Hudson commented on HBASE-5736:
---

Integrated in HBase-TRUNK #2725 (See 
[https://builds.apache.org/job/HBase-TRUNK/2725/])
HBASE-5736 ThriftServerRunner.HbaseHandler.mutateRow() does not use 
ByteBuffer correctly (Scott Chen) (Revision 1310634)

 Result = SUCCESS
tedyu : 
Files : 
* 
/hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
* 
/hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Zhihong Yu (Commented) (JIRA)

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

Zhihong Yu commented on HBASE-5736:
---

Integrated to trunk.

Thanks for the patch, Scott.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Fix For: 0.96.0
>
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Zhihong Yu (Commented) (JIRA)

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

Zhihong Yu commented on HBASE-5736:
---

The test failure was caused by OutOfMemoryError.

Will integrate later if there is no objection.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Hadoop QA (Commented) (JIRA)

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

Hadoop QA commented on HBASE-5736:
--

-1 overall.  Here are the results of testing the latest attachment 
  
http://issues.apache.org/jira/secure/attachment/12521707/HBASE-5736.D2649.3.patch
  against trunk revision .

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 6 new or modified tests.

+1 javadoc.  The javadoc tool did not generate any warning messages.

+1 javac.  The applied patch does not increase the total number of javac 
compiler warnings.

+1 findbugs.  The patch does not introduce any new Findbugs (version 1.3.9) 
warnings.

+1 release audit.  The applied patch does not increase the total number of 
release audit warnings.

 -1 core tests.  The patch failed these unit tests:
   
org.apache.hadoop.hbase.io.hfile.TestForceCacheImportantBlocks

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1432//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1432//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1432//console

This message is automatically generated.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Scott Chen (Commented) (JIRA)

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

Scott Chen commented on HBASE-5736:
---

The test got timeout. It makes sense because I am adding more steps to the test.
Increasing the timeout should fix this.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch, 
> HBASE-5736.D2649.3.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-06 Thread Scott Chen (Commented) (JIRA)

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

Scott Chen commented on HBASE-5736:
---

Failed test seems related. I will take a look.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-05 Thread Hadoop QA (Commented) (JIRA)

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

Hadoop QA commented on HBASE-5736:
--

-1 overall.  Here are the results of testing the latest attachment 
  
http://issues.apache.org/jira/secure/attachment/12521632/HBASE-5736.D2649.2.patch
  against trunk revision .

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 6 new or modified tests.

+1 javadoc.  The javadoc tool did not generate any warning messages.

+1 javac.  The applied patch does not increase the total number of javac 
compiler warnings.

+1 findbugs.  The patch does not introduce any new Findbugs (version 1.3.9) 
warnings.

+1 release audit.  The applied patch does not increase the total number of 
release audit warnings.

 -1 core tests.  The patch failed these unit tests:
   
org.apache.hadoop.hbase.regionserver.TestSplitTransactionOnCluster
  org.apache.hadoop.hbase.thrift.TestThriftServerCmdLine

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1423//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1423//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/1423//console

This message is automatically generated.

> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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




[jira] [Commented] (HBASE-5736) ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly

2012-04-05 Thread Phabricator (Commented) (JIRA)

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

Phabricator commented on HBASE-5736:


tedyu has accepted the revision "HBASE-5736 [jira] 
ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly".

  If tests pass.

REVISION DETAIL
  https://reviews.facebook.net/D2649

BRANCH
  get-bytes-thrift


> ThriftServerRunner.HbaseHandler.mutateRow() does not use ByteBuffer correctly
> -
>
> Key: HBASE-5736
> URL: https://issues.apache.org/jira/browse/HBASE-5736
> Project: HBase
>  Issue Type: Bug
>Reporter: Scott Chen
>Assignee: Scott Chen
> Attachments: HBASE-5736.D2649.1.patch, HBASE-5736.D2649.2.patch
>
>
> We have fixed similar bug in
> https://issues.apache.org/jira/browse/HBASE-5507
> It uses ByteBuffer.array() to read the ByteBuffer.
> This will ignore the offset return the whole underlying byte array.
> The bug can be triggered by using framed Transport thrift servers.

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