[GitHub] [hbase] Apache-HBase commented on pull request #2839: HBASE-25457 Possible race in AsyncConnectionImpl between getChoreServ…

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2839: URL: https://github.com/apache/hbase/pull/2839#issuecomment-753820098 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2839: HBASE-25457 Possible race in AsyncConnectionImpl between getChoreServ…

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2839: URL: https://github.com/apache/hbase/pull/2839#issuecomment-753816050 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

[GitHub] [hbase] Apache-HBase commented on pull request #2839: HBASE-25457 Possible race in AsyncConnectionImpl between getChoreServ…

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2839: URL: https://github.com/apache/hbase/pull/2839#issuecomment-753815181 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m

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

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2837: URL: https://github.com/apache/hbase/pull/2837#issuecomment-753812742 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m

[GitHub] [hbase] Apache9 opened a new pull request #2839: HBASE-25457 Possible race in AsyncConnectionImpl between getChoreServ…

2021-01-03 Thread GitBox
Apache9 opened a new pull request #2839: URL: https://github.com/apache/hbase/pull/2839 …ice and close This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [hbase] Apache-HBase commented on pull request #2838: HBASE-25458 HRegion methods cleanup

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2838: URL: https://github.com/apache/hbase/pull/2838#issuecomment-753800310 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 4m

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

2021-01-03 Thread GitBox
cuibo01 commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-753792850 thank all for reviewing the code This is an automated message from the Apache Git Service. To respond to the

[jira] [Updated] (HBASE-25457) Possible race in AsyncConnectionImpl between getChoreService and close

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25457?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duo Zhang updated HBASE-25457: -- Component/s: Client > Possible race in AsyncConnectionImpl between getChoreService and close >

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

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2837: URL: https://github.com/apache/hbase/pull/2837#issuecomment-753783380 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

[jira] [Work started] (HBASE-25458) HRegion methods cleanup

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25458?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on HBASE-25458 started by Duo Zhang. - > HRegion methods cleanup > --- > > Key:

[jira] [Assigned] (HBASE-25457) Possible race in AsyncConnectionImpl between getChoreService and close

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25457?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duo Zhang reassigned HBASE-25457: - Assignee: Duo Zhang > Possible race in AsyncConnectionImpl between getChoreService and close >

[jira] [Assigned] (HBASE-25458) HRegion methods cleanup

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25458?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duo Zhang reassigned HBASE-25458: - Assignee: Duo Zhang > HRegion methods cleanup > --- > >

[jira] [Work started] (HBASE-25457) Possible race in AsyncConnectionImpl between getChoreService and close

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25457?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on HBASE-25457 started by Duo Zhang. - > Possible race in AsyncConnectionImpl between getChoreService and close >

[GitHub] [hbase] Apache9 opened a new pull request #2838: HBASE-25458 HRegion methods cleanup

2021-01-03 Thread GitBox
Apache9 opened a new pull request #2838: URL: https://github.com/apache/hbase/pull/2838 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

[jira] [Updated] (HBASE-25458) HRegion methods cleanup

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25458?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Duo Zhang updated HBASE-25458: -- Environment: (was: HRegi) > HRegion methods cleanup > --- > >

[jira] [Commented] (HBASE-25435) Slow metric value can be configured

2021-01-03 Thread Zheng Wang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257966#comment-17257966 ] Zheng Wang commented on HBASE-25435: Backported to branch2.3+, with a bit conflict fixing. > Slow

[jira] [Commented] (HBASE-25445) SplitWALRemoteProcedure failed to archive split WAL

2021-01-03 Thread Anjan Das (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25445?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257953#comment-17257953 ] Anjan Das commented on HBASE-25445: --- [~mokai87] Yes, I have started working on the fix and I'm testing

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

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2837: URL: https://github.com/apache/hbase/pull/2837#issuecomment-753749742 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

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

2021-01-03 Thread GitBox
leyangyueshan commented on pull request #2837: URL: https://github.com/apache/hbase/pull/2837#issuecomment-753742620 #2822 closed unexpected,so i open a new pr #2837 This is an automated message from the Apache Git

[GitHub] [hbase] leyangyueshan opened a new pull request #2837: HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete…

2021-01-03 Thread GitBox
leyangyueshan opened a new pull request #2837: URL: https://github.com/apache/hbase/pull/2837 … & updatePut This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[jira] [Commented] (HBASE-25445) SplitWALRemoteProcedure failed to archive split WAL

2021-01-03 Thread mokai (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25445?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257925#comment-17257925 ] mokai commented on HBASE-25445: --- [~dasanjan1296] Have you started the fix? I'm testing the fix on my

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

2021-01-03 Thread GitBox
leyangyueshan closed pull request #2822: URL: https://github.com/apache/hbase/pull/2822 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

[jira] [Commented] (HBASE-25439) Add BYTE unit in PrettyPrinter.Unit

2021-01-03 Thread Baiqiang Zhao (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257913#comment-17257913 ] Baiqiang Zhao commented on HBASE-25439: --- Thanks for review [~stack]. Will give a PR for branch-2.

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

2021-01-03 Thread GitBox
leyangyueshan commented on pull request #2822: URL: https://github.com/apache/hbase/pull/2822#issuecomment-753730948 > @leyangyueshan there is a conflict still? See above. sorry,rebase now This is an automated message

[jira] [Updated] (HBASE-25431) MAX_FILESIZE and MEMSTORE_FLUSHSIZE should not be set negative number

2021-01-03 Thread Baiqiang Zhao (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25431?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Baiqiang Zhao updated HBASE-25431: -- Description: Before create or alter table, we will do sanityCheck in TableDescriptorChecker.

[jira] [Commented] (HBASE-25431) MAX_FILESIZE and MEMSTORE_FLUSHSIZE should not be set negative number

2021-01-03 Thread Baiqiang Zhao (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257906#comment-17257906 ] Baiqiang Zhao commented on HBASE-25431: --- Yes, the original logic is use maxFileSizeLowerLimit

[GitHub] [hbase] Apache9 commented on pull request #2714: HBASE-25333 Add a yetus check to disable usage of VisibleForTesting a…

2021-01-03 Thread GitBox
Apache9 commented on pull request #2714: URL: https://github.com/apache/hbase/pull/2714#issuecomment-753726484 Oh, it is used to address the concern that guava may remove it in the future? Sounds good to me. This is an

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

2021-01-03 Thread shenshengli (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257899#comment-17257899 ] shenshengli commented on HBASE-25449: - {quote}Do you mean, users could forget to set it in hbase and

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

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25453?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257897#comment-17257897 ] Duo Zhang commented on HBASE-25453: --- For ResultScanner, on master branch, it is something like the

[GitHub] [hbase] Apache9 commented on pull request #2714: HBASE-25333 Add a yetus check to disable usage of VisibleForTesting a…

2021-01-03 Thread GitBox
Apache9 commented on pull request #2714: URL: https://github.com/apache/hbase/pull/2714#issuecomment-753720995 > How about we add our own annotation to mark things visible for testing into the `hbase-annotations` module? I‘m OK with this approach but what's the difference with

[jira] [Commented] (HBASE-24751) Display Task completion time and/or processing duration on Web UI

2021-01-03 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-24751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257823#comment-17257823 ] Hudson commented on HBASE-24751: Results for branch branch-2 [build #145 on

[GitHub] [hbase] saintstack commented on a change in pull request #2801: HBASE-25433 There is no limit on the table name length when creating a table

2021-01-03 Thread GitBox
saintstack commented on a change in pull request #2801: URL: https://github.com/apache/hbase/pull/2801#discussion_r551034111 ## File path: hbase-common/src/main/java/org/apache/hadoop/hbase/TableName.java ## @@ -242,6 +255,9 @@ public static void isLegalNamespaceName(final

[jira] [Resolved] (HBASE-25439) Add BYTE unit in PrettyPrinter.Unit

2021-01-03 Thread Michael Stack (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Stack resolved HBASE-25439. --- Fix Version/s: 3.0.0-alpha-1 Hadoop Flags: Reviewed Resolution: Fixed Merged to

[GitHub] [hbase] saintstack merged pull request #2812: HBASE-25439 Add BYTE unit in PrettyPrinter.Unit

2021-01-03 Thread GitBox
saintstack merged pull request #2812: URL: https://github.com/apache/hbase/pull/2812 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-25450) The parameter "hbase.bucketcache.size" is misdescribed

2021-01-03 Thread Michael Stack (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257820#comment-17257820 ] Michael Stack commented on HBASE-25450: --- Merged to branch-2 and master. > The parameter

[jira] [Resolved] (HBASE-25450) The parameter "hbase.bucketcache.size" is misdescribed

2021-01-03 Thread Michael Stack (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25450?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Stack resolved HBASE-25450. --- Fix Version/s: 2.5.0 3.0.0-alpha-1 Hadoop Flags: Reviewed

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

2021-01-03 Thread GitBox
saintstack commented on pull request #2821: URL: https://github.com/apache/hbase/pull/2821#issuecomment-753649734 Merged. Test failure unrelated to doc changes. This is an automated message from the Apache Git Service. To

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

2021-01-03 Thread GitBox
saintstack merged pull request #2821: URL: https://github.com/apache/hbase/pull/2821 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-25453) Add trace support for scan operation

2021-01-03 Thread Michael Stack (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25453?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257796#comment-17257796 ] Michael Stack commented on HBASE-25453: --- Can you enforce a limit? Span count? On each #next? >

[GitHub] [hbase] busbey commented on pull request #2714: HBASE-25333 Add a yetus check to disable usage of VisibleForTesting a…

2021-01-03 Thread GitBox
busbey commented on pull request #2714: URL: https://github.com/apache/hbase/pull/2714#issuecomment-753636244 How about we add our own annotation to mark things visible for testing into the `hbase-annotations` module? This

[GitHub] [hbase] Apache9 commented on pull request #2714: HBASE-25333 Add a yetus check to disable usage of VisibleForTesting a…

2021-01-03 Thread GitBox
Apache9 commented on pull request #2714: URL: https://github.com/apache/hbase/pull/2714#issuecomment-753617483 Any other concerns here @apurtell ? When implementing HBASE-25458, I found that it is a pain that we do not have VisibleForTesting annotation. For example, this

[jira] [Commented] (HBASE-25437) Update refguide RC section; point at the dev-tools/create-releases/README.txt rather than repeat its content

2021-01-03 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257756#comment-17257756 ] Hudson commented on HBASE-25437: Results for branch master [build #173 on

[jira] [Commented] (HBASE-24751) Display Task completion time and/or processing duration on Web UI

2021-01-03 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-24751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257754#comment-17257754 ] Hudson commented on HBASE-24751: Results for branch master [build #173 on

[jira] [Commented] (HBASE-25438) Update create-release mvn in Dockerfile; its 3.6.0; make it 3.6.3

2021-01-03 Thread Hudson (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257755#comment-17257755 ] Hudson commented on HBASE-25438: Results for branch master [build #173 on

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

2021-01-03 Thread Duo Zhang (Jira)
[ https://issues.apache.org/jira/browse/HBASE-25453?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17257731#comment-17257731 ] Duo Zhang commented on HBASE-25453: --- Was thinking of add a span in AsyncClientScanner to be used as

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

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-753584637 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m

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

2021-01-03 Thread GitBox
Apache-HBase commented on pull request #2824: URL: https://github.com/apache/hbase/pull/2824#issuecomment-753584020 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m