[GitHub] [hbase] Apache-HBase commented on pull request #2836: HBASE-25456:branch-2.3 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2836: URL: https://github.com/apache/hbase/pull/2836#issuecomment-752880991 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] cuibo01 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752878534 According to the JVM specifications, the subsequent actions of OOM are "undefined behaviors", that is, after OOM occurs, it is uncertain what must be or will not be done.

[GitHub] [hbase] lujiefsi commented on pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi commented on pull request #2834: URL: https://github.com/apache/hbase/pull/2834#issuecomment-752876807 > ``` > [ERROR]

[GitHub] [hbase] Apache-HBase commented on pull request #2833: HBASE-25456:add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2833: URL: https://github.com/apache/hbase/pull/2833#issuecomment-752872413 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2835: HBASE-25456:branch-2.4 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2835: URL: https://github.com/apache/hbase/pull/2835#issuecomment-752868239 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] virajjasani commented on pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
virajjasani commented on pull request #2834: URL: https://github.com/apache/hbase/pull/2834#issuecomment-752868087 ``` [ERROR]

[GitHub] [hbase] Apache-HBase commented on pull request #2836: HBASE-25456:branch-2.3 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2836: URL: https://github.com/apache/hbase/pull/2836#issuecomment-752868049 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2836: HBASE-25456:branch-2.3 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2836: URL: https://github.com/apache/hbase/pull/2836#issuecomment-752866837 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2834: URL: https://github.com/apache/hbase/pull/2834#issuecomment-752866471 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2835: HBASE-25456:branch-2.4 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2835: URL: https://github.com/apache/hbase/pull/2835#issuecomment-752866283 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2835: HBASE-25456:branch-2.4 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2835: URL: https://github.com/apache/hbase/pull/2835#issuecomment-752866192 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m

[GitHub] [hbase] Apache-HBase commented on pull request #2836: HBASE-25456:branch-2.3 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2836: URL: https://github.com/apache/hbase/pull/2836#issuecomment-752866160 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2834: URL: https://github.com/apache/hbase/pull/2834#issuecomment-752865981 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2834: URL: https://github.com/apache/hbase/pull/2834#issuecomment-752864964 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] lujiefsi commented on pull request #2833: HBASE-25456:add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi commented on pull request #2833: URL: https://github.com/apache/hbase/pull/2833#issuecomment-752862335 > +1, keep backports ready? Thanks done. This is an automated message from the Apache Git Service. To

[GitHub] [hbase] lujiefsi opened a new pull request #2836: HBASE-25456:branch-2.3 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2836: URL: https://github.com/apache/hbase/pull/2836 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] lujiefsi opened a new pull request #2835: HBASE-25456:branch-2.4 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2835: URL: https://github.com/apache/hbase/pull/2835 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] lujiefsi opened a new pull request #2834: HBASE-25456:branch-2 add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2834: URL: https://github.com/apache/hbase/pull/2834 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] leyangyueshan commented on a change in pull request #2823: HBASE-25435 Slow metric value can be configured

2020-12-30 Thread GitBox
leyangyueshan commented on a change in pull request #2823: URL: https://github.com/apache/hbase/pull/2823#discussion_r550407297 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.java ## @@ -53,6 +54,8 @@ private Timer

[GitHub] [hbase] Apache-HBase commented on pull request #2821: Shenshengli hbase-25450 The parameter "hbase.bucketcache.size" is misdescribed

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2821: URL: https://github.com/apache/hbase/pull/2821#issuecomment-752860261 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] virajjasani commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
virajjasani commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752850561 My bad, I missed it. Build results look good now. This is an automated message from the Apache Git Service.

[GitHub] [hbase] Apache-HBase commented on pull request #2833: HBASE-25456:add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2833: URL: https://github.com/apache/hbase/pull/2833#issuecomment-752846206 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Resolved] (HBASE-25422) update_all_config should not be executed by non-admin user!!!

2020-12-30 Thread lujie (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] lujie resolved HBASE-25422. --- Resolution: Duplicate > update_all_config should not be executed by non-admin user!!! >

[GitHub] [hbase] lujiefsi opened a new pull request #2833: HBASE-25456:add security check for setRegionStateInMeta

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2833: URL: https://github.com/apache/hbase/pull/2833 @virajjasani it seems that we miss setRegionStateInMeta, fix it and will backport after review! This is an automated message from the

[GitHub] [hbase] Apache-HBase commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752836988 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752836035 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] cuibo01 edited a comment on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 edited a comment on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752833905 when throw "unable to create new native thread", the Thread does not run, jvm does not call uncaughtExceptionHandler

[GitHub] [hbase] cuibo01 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752833905 when throw "unable to create new native thread", the Thread does not run, uncaughtExceptionHandler is invalid.

[jira] [Updated] (HBASE-25456) setRegionStateInMeta need security check

2020-12-30 Thread lujie (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] lujie updated HBASE-25456: -- Description: [Viraj Jasani|https://issues.apache.org/jira/secure/ViewProfile.jspa?name=vjasani]  it seems

[jira] [Updated] (HBASE-25456) setRegionStateInMeta need security check

2020-12-30 Thread lujie (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] lujie updated HBASE-25456: -- Description: [Viraj Jasani|https://issues.apache.org/jira/secure/ViewProfile.jspa?name=vjasani]  it seems

[jira] [Updated] (HBASE-25456) setRegionStateInMeta need security check

2020-12-30 Thread lujie (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] lujie updated HBASE-25456: -- Description: @ > setRegionStateInMeta need security check > > >

[jira] [Created] (HBASE-25456) setRegionStateInMeta need security check

2020-12-30 Thread lujie (Jira)
lujie created HBASE-25456: - Summary: setRegionStateInMeta need security check Key: HBASE-25456 URL: https://issues.apache.org/jira/browse/HBASE-25456 Project: HBase Issue Type: Bug

[GitHub] [hbase] Apache9 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
Apache9 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752825371 > if we don't have a better way to solve the issue or circumvent the issue, we can fix timeoutThread first, and at least the Proc is correct. OK. Then since TimeoutExecutor

[GitHub] [hbase] Apache-HBase commented on pull request #2821: Shenshengli hbase-25450 The parameter "hbase.bucketcache.size" is misdescribed

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2821: URL: https://github.com/apache/hbase/pull/2821#issuecomment-752824012 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Commented] (HBASE-25452) Use MatcherAssert.assertThat instead of org.junit.Assert.assertThat

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256789#comment-17256789 ] Hudson commented on HBASE-25452: Results for branch branch-2.4 [build #18 on

[GitHub] [hbase] cuibo01 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752819455 if we don't have a better way to solve the issue or circumvent the issue, we can fix timeoutThread first, and at least the Proc is correct.

[GitHub] [hbase] cuibo01 removed a comment on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 removed a comment on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752817713 According to the communication with the maintenance, this problem that jvm does not exit exists in many OS components. No better solution is available.

[GitHub] [hbase] cuibo01 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752817713 According to the communication with the maintenance, this problem that jvm does not exit exists in many OS components. No better solution is available.

[GitHub] [hbase] cuibo01 edited a comment on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 edited a comment on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752816204 analyze and verify the java native code, some OMME can not cause the JVM to exit, like "java.lang.OutOfMemoryError: Direct buffer memory", "java.lang.OutOfMemoryError:

[GitHub] [hbase] cuibo01 edited a comment on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 edited a comment on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752816204 analyze and verify the java native code, some OMME can not be the JVM exit, like "java.lang.OutOfMemoryError: Direct buffer memory", "java.lang.OutOfMemoryError: unable to

[GitHub] [hbase] Apache-HBase commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752816365 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] cuibo01 commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752816204 analyze and verify the java native code, some OMME do not cause the JVM to exit, like "java.lang.OutOfMemoryError: Direct buffer memory", "java.lang.OutOfMemoryError: unable to

[GitHub] [hbase] Apache-HBase commented on pull request #2821: Shenshengli hbase-25450 The parameter "hbase.bucketcache.size" is misdescribed

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2821: URL: https://github.com/apache/hbase/pull/2821#issuecomment-752816174 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Commented] (HBASE-25449) 'dfs.client.read.shortcircuit' should not be set in hbase-default.xml

2020-12-30 Thread Michael Stack (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256774#comment-17256774 ] Michael Stack commented on HBASE-25449: --- bq. this can easily be ignored Do you mean, users

[GitHub] [hbase] Apache9 commented on a change in pull request #2825: HBASE-25451 Upgrade commons-io to 2.8.0

2020-12-30 Thread GitBox
Apache9 commented on a change in pull request #2825: URL: https://github.com/apache/hbase/pull/2825#discussion_r550372045 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java ## @@ -832,7 +832,7 @@ private void

[GitHub] [hbase] saintstack commented on pull request #2821: Shenshengli hbase-25450 The parameter "hbase.bucketcache.size" is misdescribed

2020-12-30 Thread GitBox
saintstack commented on pull request #2821: URL: https://github.com/apache/hbase/pull/2821#issuecomment-752810577 Thanks for the feedback @anoopsjohn and @shenshengli . Let me rerun unit tests to make sure the PR is not responsible for test failures.

[GitHub] [hbase] saintstack commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
saintstack commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752809897 @virajjasani there is an explanation in the above... check it out. Let me rerun the tests. The failures look like flakies. Rerunning to make sure.

[GitHub] [hbase] saintstack commented on pull request #2824: HBASE-25447 remoteProc is suspended due to OOM ERROR

2020-12-30 Thread GitBox
saintstack commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-752809514 Agree. If OOME, process should go down... die. We can't keep going. It is a fatal condition. This is an

[GitHub] [hbase] saintstack commented on a change in pull request #2825: HBASE-25451 Upgrade commons-io to 2.8.0

2020-12-30 Thread GitBox
saintstack commented on a change in pull request #2825: URL: https://github.com/apache/hbase/pull/2825#discussion_r550368884 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java ## @@ -832,7 +832,7 @@ private void

[GitHub] [hbase] saintstack commented on a change in pull request #2828: HBASE-25454 Add trace support for connection registry

2020-12-30 Thread GitBox
saintstack commented on a change in pull request #2828: URL: https://github.com/apache/hbase/pull/2828#discussion_r550367007 ## File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/MasterRegistry.java ## @@ -18,6 +18,7 @@ package

[jira] [Resolved] (HBASE-25441) add security check for some APIs in RSRpcServices

2020-12-30 Thread Viraj Jasani (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viraj Jasani resolved HBASE-25441. -- Hadoop Flags: Reviewed Resolution: Fixed Thanks for the contribution [~xiaoheipangzi].

[GitHub] [hbase] virajjasani merged pull request #2832: HBASE-25441:branch-1 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
virajjasani merged pull request #2832: URL: https://github.com/apache/hbase/pull/2832 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [hbase] virajjasani merged pull request #2829: HBASE-25441:branch-2.3 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
virajjasani merged pull request #2829: URL: https://github.com/apache/hbase/pull/2829 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [hbase] virajjasani merged pull request #2830: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
virajjasani merged pull request #2830: URL: https://github.com/apache/hbase/pull/2830 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [hbase] virajjasani merged pull request #2831: HBASE-25441:branch-2 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
virajjasani merged pull request #2831: URL: https://github.com/apache/hbase/pull/2831 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [hbase] virajjasani merged pull request #2810: HBASE-25441 : add security check for RSRpcSever

2020-12-30 Thread GitBox
virajjasani merged pull request #2810: URL: https://github.com/apache/hbase/pull/2810 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[jira] [Commented] (HBASE-25452) Use MatcherAssert.assertThat instead of org.junit.Assert.assertThat

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256648#comment-17256648 ] Hudson commented on HBASE-25452: Results for branch branch-2 [build #141 on

[jira] [Commented] (HBASE-25379) Make retry pause time configurable for regionserver short operation RPC (reportRegionStateTransition/reportProcedureDone)

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256647#comment-17256647 ] Hudson commented on HBASE-25379: Results for branch branch-2 [build #141 on

[GitHub] [hbase] Apache-HBase commented on pull request #2831: HBASE-25441:branch-2 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2831: URL: https://github.com/apache/hbase/pull/2831#issuecomment-752714955 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2830: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2830: URL: https://github.com/apache/hbase/pull/2830#issuecomment-752713559 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2832: HBASE-25441:branch-1 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2832: URL: https://github.com/apache/hbase/pull/2832#issuecomment-752712202 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 7m

[GitHub] [hbase] Apache-HBase commented on pull request #2830: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2830: URL: https://github.com/apache/hbase/pull/2830#issuecomment-752711824 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2829: HBASE-25441:branch-2.3 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2829: URL: https://github.com/apache/hbase/pull/2829#issuecomment-752709582 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2831: HBASE-25441:branch-2 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2831: URL: https://github.com/apache/hbase/pull/2831#issuecomment-752709022 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2829: HBASE-25441:branch-2.3 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2829: URL: https://github.com/apache/hbase/pull/2829#issuecomment-752704863 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Commented] (HBASE-25452) Use MatcherAssert.assertThat instead of org.junit.Assert.assertThat

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256608#comment-17256608 ] Hudson commented on HBASE-25452: Results for branch branch-2.3 [build #134 on

[GitHub] [hbase] Apache-HBase commented on pull request #2830: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2830: URL: https://github.com/apache/hbase/pull/2830#issuecomment-752672338 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m

[GitHub] [hbase] Apache-HBase commented on pull request #2831: HBASE-25441:branch-2 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2831: URL: https://github.com/apache/hbase/pull/2831#issuecomment-752669473 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2829: HBASE-25441:branch-2.3 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2829: URL: https://github.com/apache/hbase/pull/2829#issuecomment-752667099 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2828: HBASE-25454 Add trace support for connection registry

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2828: URL: https://github.com/apache/hbase/pull/2828#issuecomment-752664746 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] lujiefsi opened a new pull request #2832: HBASE-25441:branch-1 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2832: URL: https://github.com/apache/hbase/pull/2832 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] Apache-HBase commented on pull request #2828: HBASE-25454 Add trace support for connection registry

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2828: URL: https://github.com/apache/hbase/pull/2828#issuecomment-752661157 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2828: HBASE-25454 Add trace support for connection registry

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2828: URL: https://github.com/apache/hbase/pull/2828#issuecomment-752660646 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] lujiefsi opened a new pull request #2831: HBASE-25441:branch-2 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2831: URL: https://github.com/apache/hbase/pull/2831 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] lujiefsi opened a new pull request #2830: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2830: URL: https://github.com/apache/hbase/pull/2830 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] lujiefsi opened a new pull request #2829: HBASE-25441:branch-2.3 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2829: URL: https://github.com/apache/hbase/pull/2829 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] Apache9 opened a new pull request #2828: HBASE-25454 Add trace support for connection registry

2020-12-30 Thread GitBox
Apache9 opened a new pull request #2828: URL: https://github.com/apache/hbase/pull/2828 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] lujiefsi closed pull request #2827: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi closed pull request #2827: URL: https://github.com/apache/hbase/pull/2827 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [hbase] Apache-HBase commented on pull request #2827: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2827: URL: https://github.com/apache/hbase/pull/2827#issuecomment-752649302 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [hbase] lujiefsi opened a new pull request #2827: HBASE-25441:branch-2.4 add security check for some APIs in RSRpcServices

2020-12-30 Thread GitBox
lujiefsi opened a new pull request #2827: URL: https://github.com/apache/hbase/pull/2827 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [hbase] Apache9 commented on a change in pull request #2825: HBASE-25451 Upgrade commons-io to 2.8.0

2020-12-30 Thread GitBox
Apache9 commented on a change in pull request #2825: URL: https://github.com/apache/hbase/pull/2825#discussion_r550210345 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java ## @@ -832,7 +832,7 @@ private void

[GitHub] [hbase] lujiefsi edited a comment on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
lujiefsi edited a comment on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752637590 > backport I will do it as soon as possiable This is an automated message from the Apache Git

[GitHub] [hbase] lujiefsi commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
lujiefsi commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752637590 > backport I wll do it as soon as possiable This is an automated message from the Apache Git Service. To

[GitHub] [hbase] virajjasani commented on pull request #2822: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2020-12-30 Thread GitBox
virajjasani commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-752635389 > How does this relate to HBASE-25435 and #2823 ? Does this have to go in before #2823? You add a bunch of hardcoding in here and then over in #2823 you try and make it

[jira] [Updated] (HBASE-25441) add security check for some APIs in RSRpcServices

2020-12-30 Thread Viraj Jasani (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viraj Jasani updated HBASE-25441: - Fix Version/s: 2.4.1 2.5.0 2.3.4 1.7.0

[GitHub] [hbase] virajjasani commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
virajjasani commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752633806 @lujiefsi Can you keep backport PRs ready? For changes this sensitive (where you already discovered the importance of overriding `isOnline()` in HMaster), we better get QA

[jira] [Updated] (HBASE-25454) Add trace support for connection registry

2020-12-30 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25454?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duo Zhang updated HBASE-25454: -- Component/s: tracing > Add trace support for connection registry >

[jira] [Commented] (HBASE-25452) Use MatcherAssert.assertThat instead of org.junit.Assert.assertThat

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256512#comment-17256512 ] Hudson commented on HBASE-25452: Results for branch master [build #168 on

[jira] [Commented] (HBASE-25379) Make retry pause time configurable for regionserver short operation RPC (reportRegionStateTransition/reportProcedureDone)

2020-12-30 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17256511#comment-17256511 ] Hudson commented on HBASE-25379: Results for branch master [build #168 on

[GitHub] [hbase] lujiefsi commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
lujiefsi commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752420867 explain why UT fails and why make changes: (1) rpcPreCheck will first check whether server is online by _'isOnline'_ method, but HMaster do not have such method, so it always

[jira] [Updated] (HBASE-25434) SlowDelete & SlowPut metric value should use updateDelete & updatePut

2020-12-30 Thread xijiawen (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25434?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xijiawen updated HBASE-25434: - Priority: Major (was: Critical) > SlowDelete & SlowPut metric value should use updateDelete &

[GitHub] [hbase] Apache-HBase commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752415776 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[GitHub] [hbase] Apache-HBase commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752415001 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Created] (HBASE-25455) Add trace support for HRegion read/write operation

2020-12-30 Thread Duo Zhang (Jira)
Duo Zhang created HBASE-25455: - Summary: Add trace support for HRegion read/write operation Key: HBASE-25455 URL: https://issues.apache.org/jira/browse/HBASE-25455 Project: HBase Issue Type:

[jira] [Created] (HBASE-25454) Add trace support for connection registry

2020-12-30 Thread Duo Zhang (Jira)
Duo Zhang created HBASE-25454: - Summary: Add trace support for connection registry Key: HBASE-25454 URL: https://issues.apache.org/jira/browse/HBASE-25454 Project: HBase Issue Type: Sub-task

[jira] [Created] (HBASE-25453) Add trace support for scan operation

2020-12-30 Thread Duo Zhang (Jira)
Duo Zhang created HBASE-25453: - Summary: Add trace support for scan operation Key: HBASE-25453 URL: https://issues.apache.org/jira/browse/HBASE-25453 Project: HBase Issue Type: Sub-task

[GitHub] [hbase] Apache-HBase commented on pull request #2810: HBASE-25441:add security check for RSRpcSever

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2810: URL: https://github.com/apache/hbase/pull/2810#issuecomment-752378058 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2813: HBASE-23898 Add trace support for simple apis in async client

2020-12-30 Thread GitBox
Apache-HBase commented on pull request #2813: URL: https://github.com/apache/hbase/pull/2813#issuecomment-752366492 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m