[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249865#comment-13249865
]
stack commented on HBASE-5727:
--
+1 on patch. On commit should add curlies around the log
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249917#comment-13249917
]
Hadoop QA commented on HBASE-5727:
--
-1 overall. Here are the results of testing the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249931#comment-13249931
]
Hudson commented on HBASE-5727:
---
Integrated in HBase-TRUNK #2731 (See
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249934#comment-13249934
]
Hudson commented on HBASE-5727:
---
Integrated in HBase-TRUNK-security #164 (See
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249954#comment-13249954
]
Zhihong Yu commented on HBASE-5727:
---
Nicolas checked in HBASE-5335 a little earlier than
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249967#comment-13249967
]
Devaraj Das commented on HBASE-5727:
But this isn't related to the patch committed,
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249968#comment-13249968
]
Zhihong Yu commented on HBASE-5727:
---
@Devaraj:
I should have explained in clearer terms.
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249975#comment-13249975
]
Devaraj Das commented on HBASE-5727:
bq. Another option is to ask Nicolas to put the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249980#comment-13249980
]
stack commented on HBASE-5727:
--
Or just add it back w/ addendum on this patch.
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249981#comment-13249981
]
stack commented on HBASE-5727:
--
Sorry. Scratch the above suggestion.
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249994#comment-13249994
]
Zhihong Yu commented on HBASE-5727:
---
I tend to agree with Devaraj in that HBASE-5335
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249998#comment-13249998
]
stack commented on HBASE-5727:
--
@Ted No. Thats why we run a secure build up on jenkins to
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250013#comment-13250013
]
Andrew Purtell commented on HBASE-5727:
---
Removing getConf() was an action that had
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250028#comment-13250028
]
Nicolas Spiegelberg commented on HBASE-5727:
I agree with Stack, this isn't a
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250037#comment-13250037
]
Andrew Purtell commented on HBASE-5727:
---
bq. If your coprocessor uses the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250077#comment-13250077
]
stack commented on HBASE-5727:
--
Patch looks right to me; its providing the region context
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250140#comment-13250140
]
Zhihong Yu commented on HBASE-5727:
---
If there is no objection, I will integrate the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250348#comment-13250348
]
Zhihong Yu commented on HBASE-5727:
---
Integrated addendum to trunk.
Jenkins is having
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249612#comment-13249612
]
Ted Yu commented on HBASE-5727:
---
TestRowProcessorEndpoint in security profile passed
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249616#comment-13249616
]
Hadoop QA commented on HBASE-5727:
--
-1 overall. Here are the results of testing the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249289#comment-13249289
]
Zhihong Yu commented on HBASE-5727:
---
One test consistently fails in secure build:
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249292#comment-13249292
]
Devaraj Das commented on HBASE-5727:
Thanks, Zhihong. Will take a look.
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247329#comment-13247329
]
Devaraj Das commented on HBASE-5727:
I will get to it today. Sorry that I broke the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247469#comment-13247469
]
Andrew Purtell commented on HBASE-5727:
---
I commented over on HBASE-5451: Subclassing
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247474#comment-13247474
]
Andrew Purtell commented on HBASE-5727:
---
So, to be clear, I mean as part of this
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247507#comment-13247507
]
Andrew Purtell commented on HBASE-5727:
---
@Stack, @Devaraj: Do you want to try
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247511#comment-13247511
]
stack commented on HBASE-5727:
--
@Andrew Please advise. I'm not up on whats involved. What
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247512#comment-13247512
]
Devaraj Das commented on HBASE-5727:
Yes, I agree that we should merge the
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247517#comment-13247517
]
Andrew Purtell commented on HBASE-5727:
---
@Stack. Fixing the build as an interim step
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247520#comment-13247520
]
Andrew Purtell commented on HBASE-5727:
---
bq. Pushing through Hudson. Let me know if
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247662#comment-13247662
]
stack commented on HBASE-5727:
--
I committed this patch and forced build of secure trunk.
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247664#comment-13247664
]
Hudson commented on HBASE-5727:
---
Integrated in HBase-TRUNK-security #158 (See
[
https://issues.apache.org/jira/browse/HBASE-5727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13247719#comment-13247719
]
Hudson commented on HBASE-5727:
---
Integrated in HBase-TRUNK #2715 (See
33 matches
Mail list logo