[GitHub] [hbase] tedyu commented on pull request #1635: HBASE-23996 Many split related metrics were present RS side but after split is moved to Master, these metrics are lost.

2020-08-09 Thread GitBox
tedyu commented on pull request #1635: URL: https://github.com/apache/hbase/pull/1635#issuecomment-671159088 +1 Can rerun tests to see if they pass. This is an automated message from the Apache Git Service. To respond

[GitHub] [hbase] tedyu commented on pull request #1635: HBASE-23996 Many split related metrics were present RS side but after split is moved to Master, these metrics are lost.

2020-08-08 Thread GitBox
tedyu commented on pull request #1635: URL: https://github.com/apache/hbase/pull/1635#issuecomment-670983640 Yes, please. This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [hbase] tedyu commented on pull request #1635: HBASE-23996 Many split related metrics were present RS side but after split is moved to Master, these metrics are lost.

2020-08-08 Thread GitBox
tedyu commented on pull request #1635: URL: https://github.com/apache/hbase/pull/1635#issuecomment-670979637 Thanks for the PR. I am a bit curious: previously there were incrSplitRequest and the success count. In the PR, there is also failure count which can be deduced from the other

[GitHub] [hbase] tedyu commented on pull request #1635: HBASE-23996 Many split related metrics were present RS side but after split is moved to Master, these metrics are lost.

2020-07-30 Thread GitBox
tedyu commented on pull request #1635: URL: https://github.com/apache/hbase/pull/1635#issuecomment-666074959 Can the checkstyle warnings for hbase-hadoop-compat be addressed ? This is an automated message from the Apache Git