[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-29 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15618733#comment-15618733 ] Hudson commented on HBASE-16891: FAILURE: Integrated in Jenkins build HBase-Trunk_matrix

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-29 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15618295#comment-15618295 ] Duo Zhang commented on HBASE-16891: --- It will but it could only happen in testcase. In A

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-29 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15618205#comment-15618205 ] stack commented on HBASE-16891: --- Is this a create of a Runnable per byte written? eve

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-29 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15618033#comment-15618033 ] Duo Zhang commented on HBASE-16891: --- Thanks. Will commit later if no other objections.

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15617552#comment-15617552 ] ramkrishna.s.vasudevan commented on HBASE-16891: Am ffine and +1 if tests

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15617052#comment-15617052 ] Duo Zhang commented on HBASE-16891: --- Any concerns on the new patch? [~anoopsamjohn] [~r

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15615684#comment-15615684 ] Hadoop QA commented on HBASE-16891: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15615279#comment-15615279 ] Duo Zhang commented on HBASE-16891: --- Let me check the failed UTs. > Try copying to the

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15615257#comment-15615257 ] Hadoop QA commented on HBASE-16891: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15615057#comment-15615057 ] Duo Zhang commented on HBASE-16891: --- The implementation throws IOException which will m

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15615019#comment-15615019 ] Anoop Sam John commented on HBASE-16891: That is ok. I mean instead of java BAOS

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614999#comment-15614999 ] Duo Zhang commented on HBASE-16891: --- This is used to make the behavior the same with Fa

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614990#comment-15614990 ] Anoop Sam John commented on HBASE-16891: In AsyncFSOutputHelper {code} @Overrid

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614951#comment-15614951 ] ramkrishna.s.vasudevan commented on HBASE-16891: bq.But it does not matte

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614945#comment-15614945 ] Duo Zhang commented on HBASE-16891: --- I'm not sure but I think it will change the bb's p

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614934#comment-15614934 ] ramkrishna.s.vasudevan commented on HBASE-16891: LGTM. Just ensure that {

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614900#comment-15614900 ] Duo Zhang commented on HBASE-16891: --- It was removed by accident... I wanted to change i

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614866#comment-15614866 ] ramkrishna.s.vasudevan commented on HBASE-16891: In patch in writeMagicA

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614821#comment-15614821 ] ramkrishna.s.vasudevan commented on HBASE-16891: I think we will commit t

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614822#comment-15614822 ] Duo Zhang commented on HBASE-16891: --- Got it. Thanks. So here we should make the Output

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614818#comment-15614818 ] ramkrishna.s.vasudevan commented on HBASE-16891: The magic lies here {cod

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614811#comment-15614811 ] Anoop Sam John commented on HBASE-16891: U can see in EnsureKvEncoder {code} publ

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614798#comment-15614798 ] Duo Zhang commented on HBASE-16891: --- I do not know what you guys have done to the CellE

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614777#comment-15614777 ] ramkrishna.s.vasudevan commented on HBASE-16891: I can make the patch for

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614756#comment-15614756 ] Anoop Sam John commented on HBASE-16891: Good that we can avoid 2 copy now. But t

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614729#comment-15614729 ] ramkrishna.s.vasudevan commented on HBASE-16891: In fact if AsyncOutput c

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-28 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15614710#comment-15614710 ] ramkrishna.s.vasudevan commented on HBASE-16891: This looks good. So you

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15612582#comment-15612582 ] ramkrishna.s.vasudevan commented on HBASE-16891: Oh, now I got what you m

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15612581#comment-15612581 ] ramkrishna.s.vasudevan commented on HBASE-16891: Oh, now I got what you m

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611971#comment-15611971 ] Anoop Sam John commented on HBASE-16891: We have an OS extension mainly for writi

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611828#comment-15611828 ] Duo Zhang commented on HBASE-16891: --- I do not think this is a good idea. You should fol

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611788#comment-15611788 ] ramkrishna.s.vasudevan commented on HBASE-16891: bq.You change the semant

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611758#comment-15611758 ] Duo Zhang commented on HBASE-16891: --- You change the semantic of {{void write(int b)}} m

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611746#comment-15611746 ] Duo Zhang commented on HBASE-16891: --- Oh Codec is LimitedPrivate... And BaseEncoder is a

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611737#comment-15611737 ] ramkrishna.s.vasudevan commented on HBASE-16891: If we add new methods in

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread Duo Zhang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611706#comment-15611706 ] Duo Zhang commented on HBASE-16891: --- I think we'd better add new methods to the CellEnc

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-27 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611615#comment-15611615 ] ramkrishna.s.vasudevan commented on HBASE-16891: [~Apache9] Is it ok to c

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-24 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15604201#comment-15604201 ] ramkrishna.s.vasudevan commented on HBASE-16891: The hack that I tried he

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-23 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15600965#comment-15600965 ] ramkrishna.s.vasudevan commented on HBASE-16891: Both are doing PureJava3

[jira] [Commented] (HBASE-16891) Try copying to the Netty ByteBuf directly from the WALEdit

2016-10-21 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15596736#comment-15596736 ] stack commented on HBASE-16891: --- We doing native checksumming in both places? > Try copyin