[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636427222 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 7s | Docker mode activated. | | -0 :warning: | yetus | 0m 2s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 21s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 4m 3s | master passed | | +1 :green_heart: | compile | 0m 41s | master passed | | +1 :green_heart: | shadedjars | 6m 2s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 35s | master passed | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 14s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 45s | the patch passed | | +1 :green_heart: | compile | 0m 41s | the patch passed | | +1 :green_heart: | javac | 0m 41s | the patch passed | | +1 :green_heart: | shadedjars | 6m 2s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 34s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 27s | hbase-common in the patch passed. | | +1 :green_heart: | unit | 0m 45s | hbase-http in the patch passed. | | | | 27m 36s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux bf78431859eb 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / b4a4debdd9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/3/testReport/ | | Max. process+thread count | 418 (vs. ulimit of 12500) | | modules | C: hbase-common hbase-http U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/3/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636426636 added a missing update to hbase-default.xml 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24475) Clean up the master thread name getting in SplitLogManager and AssignmentManager
[ https://issues.apache.org/jira/browse/HBASE-24475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120414#comment-17120414 ] Zheng Wang commented on HBASE-24475: Thanks. > Clean up the master thread name getting in SplitLogManager and > AssignmentManager > > > Key: HBASE-24475 > URL: https://issues.apache.org/jira/browse/HBASE-24475 > Project: HBase > Issue Type: Improvement >Reporter: Zheng Wang >Assignee: Zheng Wang >Priority: Minor > Fix For: 3.0.0-alpha-1, 2.3.0 > > > As discussed in PR of HBASE-24451, seems just use > master.getServerName().toShortString() is enough. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1816: HBASE-24454 - Read ioErrorStartTime to local temporary variable…
Apache-HBase commented on pull request #1816: URL: https://github.com/apache/hbase/pull/1816#issuecomment-636425906 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 47s | master passed | | +1 :green_heart: | compile | 0m 56s | master passed | | +1 :green_heart: | shadedjars | 5m 31s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 37s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 29s | the patch passed | | +1 :green_heart: | compile | 0m 57s | the patch passed | | +1 :green_heart: | javac | 0m 57s | the patch passed | | +1 :green_heart: | shadedjars | 5m 34s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 36s | the patch passed | ||| _ Other Tests _ | | -1 :x: | unit | 152m 3s | hbase-server in the patch failed. | | | | 175m 59s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1816 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 0cb02824622b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/testReport/ | | Max. process+thread count | 4059 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1816: HBASE-24454 - Read ioErrorStartTime to local temporary variable…
Apache-HBase commented on pull request #1816: URL: https://github.com/apache/hbase/pull/1816#issuecomment-636425247 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 23s | master passed | | +1 :green_heart: | compile | 1m 5s | master passed | | +1 :green_heart: | shadedjars | 5m 53s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 42s | hbase-server in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 8s | the patch passed | | +1 :green_heart: | compile | 1m 5s | the patch passed | | +1 :green_heart: | javac | 1m 5s | the patch passed | | +1 :green_heart: | shadedjars | 6m 10s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 49s | hbase-server in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 142m 33s | hbase-server in the patch passed. | | | | 169m 5s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1816 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux d3ea47eacc53 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/testReport/ | | Max. process+thread count | 3972 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Resolved] (HBASE-24475) Clean up the master thread name getting in SplitLogManager and AssignmentManager
[ https://issues.apache.org/jira/browse/HBASE-24475?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Stack resolved HBASE-24475. --- Fix Version/s: 2.3.0 3.0.0-alpha-1 Hadoop Flags: Reviewed Resolution: Fixed Pushed to branch-2.3+. Thanks [~filtertip] > Clean up the master thread name getting in SplitLogManager and > AssignmentManager > > > Key: HBASE-24475 > URL: https://issues.apache.org/jira/browse/HBASE-24475 > Project: HBase > Issue Type: Improvement >Reporter: Zheng Wang >Assignee: Zheng Wang >Priority: Minor > Fix For: 3.0.0-alpha-1, 2.3.0 > > > As discussed in PR of HBASE-24451, seems just use > master.getServerName().toShortString() is enough. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] saintstack merged pull request #1810: HBASE-24475 Clean up the master thread name getting in SplitLogManage…
saintstack merged pull request #1810: URL: https://github.com/apache/hbase/pull/1810 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] saintstack commented on a change in pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
saintstack commented on a change in pull request #1811: URL: https://github.com/apache/hbase/pull/1811#discussion_r432909987 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/region/MasterRegion.java ## @@ -284,18 +284,21 @@ public static LocalRegion create(LocalRegionParams params) throws IOException { Configuration conf = new Configuration(baseConf); CommonFSUtils.setRootDir(conf, rootDir); CommonFSUtils.setWALRootDir(conf, walRootDir); -LocalRegionFlusherAndCompactor.setupConf(conf, params.flushSize(), params.flushPerChanges(), +MasterRegionFlusherAndCompactor.setupConf(conf, params.flushSize(), params.flushPerChanges(), params.flushIntervalMs()); conf.setInt(AbstractFSWAL.MAX_LOGS, params.maxWals()); if (params.useHsync() != null) { conf.setBoolean(HRegion.WAL_HSYNC_CONF_KEY, params.useHsync()); } +if (params.useMetaCellComparator() != null) { Review comment: We need to add this in here? ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/region/MasterRegion.java ## @@ -284,18 +284,21 @@ public static LocalRegion create(LocalRegionParams params) throws IOException { Configuration conf = new Configuration(baseConf); CommonFSUtils.setRootDir(conf, rootDir); CommonFSUtils.setWALRootDir(conf, walRootDir); -LocalRegionFlusherAndCompactor.setupConf(conf, params.flushSize(), params.flushPerChanges(), +MasterRegionFlusherAndCompactor.setupConf(conf, params.flushSize(), params.flushPerChanges(), params.flushIntervalMs()); conf.setInt(AbstractFSWAL.MAX_LOGS, params.maxWals()); if (params.useHsync() != null) { conf.setBoolean(HRegion.WAL_HSYNC_CONF_KEY, params.useHsync()); } +if (params.useMetaCellComparator() != null) { Review comment: This was missing? ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/region/MasterRegionFactory.java ## @@ -89,45 +82,8 @@ private static final TableDescriptor TABLE_DESC = TableDescriptorBuilder.newBuilder(TABLE_NAME) .setColumnFamily(ColumnFamilyDescriptorBuilder.of(PROC_FAMILY)).build(); - private final LocalRegion region; - - private LocalStore(LocalRegion region) { -this.region = region; - } - - public void update(UpdateLocalRegion action) throws IOException { -region.update(action); - } - - public Result get(Get get) throws IOException { -return region.get(get); - } - - public RegionScanner getScanner(Scan scan) throws IOException { -return region.getScanner(scan); - } - - public void close(boolean abort) { -region.close(abort); - } - - @VisibleForTesting - public FlushResult flush(boolean force) throws IOException { -return region.flush(force); - } - - @VisibleForTesting - public void requestRollAll() { -region.requestRollAll(); - } - - @VisibleForTesting - public void waitUntilWalRollFinished() throws InterruptedException { -region.waitUntilWalRollFinished(); - } - - public static LocalStore create(Server server) throws IOException { -LocalRegionParams params = new LocalRegionParams().server(server) Review comment: How comes we can drop these methods? 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] nyl3532016 opened a new pull request #1817: HBASE-24412 Canary support check only one column family per RegionTask
nyl3532016 opened a new pull request #1817: URL: https://github.com/apache/hbase/pull/1817 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Updated] (HBASE-24412) Canary support check only one column family per RegionTask
[ https://issues.apache.org/jira/browse/HBASE-24412?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] niuyulin updated HBASE-24412: - Summary: Canary support check only one column family per RegionTask (was: Canary support check only one column family per region) > Canary support check only one column family per RegionTask > -- > > Key: HBASE-24412 > URL: https://issues.apache.org/jira/browse/HBASE-24412 > Project: HBase > Issue Type: Improvement > Components: canary >Reporter: niuyulin >Assignee: niuyulin >Priority: Major > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] mmpataki edited a comment on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki edited a comment on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636420308 @saintstack > @mmpataki We have flakey tests. I am sorry, I ran tests from the IDE and not from mvn, so this build issue came up. > Looks good but we export protmetheus metrics even when they are not of use, not being accessed? thank you for the review, I will add a configuration to enable it (disabled by default). // I asked about this in mailing list but didn't get any responses. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636420308 @saintstack > @mmpataki We have flakey tests. I am sorry, I ran tests from the IDE and not from mvn, so this build came up. > Looks good but we export protmetheus metrics even when they are not of use, not being accessed? thank you for the review, I will add a configuration to enable it (disabled by default). // I asked about this in mailing list but didn't get any responses. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-20904) Prometheus /metrics http endpoint for monitoring integration
[ https://issues.apache.org/jira/browse/HBASE-20904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120405#comment-17120405 ] Michael Stack commented on HBASE-20904: --- What is the 'new metrics API'? Thanks. Left a question on the PR. > Prometheus /metrics http endpoint for monitoring integration > > > Key: HBASE-20904 > URL: https://issues.apache.org/jira/browse/HBASE-20904 > Project: HBase > Issue Type: New Feature > Components: metrics, monitoring >Reporter: Hari Sekhon >Priority: Major > > Feature Request to add Prometheus /metrics http endpoint for monitoring > integration: > [https://prometheus.io/docs/prometheus/latest/configuration/configuration/#%3Cscrape_config%3E] > Prometheus metrics format for that endpoint: > [https://github.com/prometheus/docs/blob/master/content/docs/instrumenting/exposition_formats.md] > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] saintstack commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
saintstack commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636419167 @mmpataki We have flakey tests. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24454) BucketCache disabled instantly before error duration toleration is reached due to timing issue
[ https://issues.apache.org/jira/browse/HBASE-24454?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120399#comment-17120399 ] Michael Stack commented on HBASE-24454: --- Added you as a contributor [~jacob.leblanc] > BucketCache disabled instantly before error duration toleration is reached > due to timing issue > -- > > Key: HBASE-24454 > URL: https://issues.apache.org/jira/browse/HBASE-24454 > Project: HBase > Issue Type: Bug > Components: BucketCache >Affects Versions: 1.4.10 > Environment: We saw this in HBase 1.4.10 (EMR 5.28.1), but I believe > the newest code still has this problem. >Reporter: Jacob LeBlanc >Assignee: Jacob LeBlanc >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > We saw an instance where BucketCache was disabled after only two IO error > were encountered at nearly the same time. The following shows all errors from > a region server log for the 2020-05-26 17:00 hour. Notice that there are no > other errors until the 17:14:50 at which time the BucketCache is disabled > because it claims duration time has exceeded 6 ms: > $ grep ERROR hbase-hbase-regionserver-ip-172-20-113-147.log.2020-05-26-17 > 2020-05-26 17:14:50,396 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,397 ERROR > [regionserver/ip-172-20-113-147.us-west-2.compute.internal/172.20.113.147:16020-BucketCacheWriter-0] > bucket.BucketCache: Failed syncing IO engine > 2020-05-26 17:14:50,400 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,401 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: IO errors duration time has exceeded 6ms, disabling > cache, please check your IOEngine > The region server is very busy so it should be constantly getting successful > reads and writes in the bucket cache (it is not as though there was some long > ago error and then no successful IO to clear the error flag). > We are running a busy EMR cluster backed by S3. A bucketcache getting > disabled is a huge performance issue because all reads must go to S3. > Looking at the code, I believe I've found a timing issue. Here is the code > for checking and setting the ioErrorStartTime (taken from BucketCache.java): > > {code:java} > /** >* Check whether we tolerate IO error this time. If the duration of IOEngine >* throwing errors exceeds ioErrorsDurationTimeTolerated, we will disable > the >* cache >*/ > private void checkIOErrorIsTolerated() { > long now = EnvironmentEdgeManager.currentTime(); > if (this.ioErrorStartTime > 0) { > if (cacheEnabled && (now - ioErrorStartTime) > > this.ioErrorsTolerationDuration) { > LOG.error("IO errors duration time has exceeded " + > ioErrorsTolerationDuration + > "ms, disabling cache, please check your IOEngine"); > disableCache(); > } > } else { > this.ioErrorStartTime = now; > } > } > {code} > > And here is the code for clearing the ioErrorStartTime when a successful read > or write is done: > {code:java} > if (this.ioErrorStartTime > 0) { > ioErrorStartTime = -1; > } > {code} > Notice that that if ioErrorStartTime is set to -1 after the first if > statement in checkIOErrorIsTolerated but before (now - ioErrorStartTime), > then (now - ioErrorStartTime) will evaluate to (now + 1) resulting in the > code thinking it has exceeded ioErrorsTolerationDuration. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Assigned] (HBASE-24454) BucketCache disabled instantly before error duration toleration is reached due to timing issue
[ https://issues.apache.org/jira/browse/HBASE-24454?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Stack reassigned HBASE-24454: - Assignee: Jacob LeBlanc (was: Michael Stack) > BucketCache disabled instantly before error duration toleration is reached > due to timing issue > -- > > Key: HBASE-24454 > URL: https://issues.apache.org/jira/browse/HBASE-24454 > Project: HBase > Issue Type: Bug > Components: BucketCache >Affects Versions: 1.4.10 > Environment: We saw this in HBase 1.4.10 (EMR 5.28.1), but I believe > the newest code still has this problem. >Reporter: Jacob LeBlanc >Assignee: Jacob LeBlanc >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > We saw an instance where BucketCache was disabled after only two IO error > were encountered at nearly the same time. The following shows all errors from > a region server log for the 2020-05-26 17:00 hour. Notice that there are no > other errors until the 17:14:50 at which time the BucketCache is disabled > because it claims duration time has exceeded 6 ms: > $ grep ERROR hbase-hbase-regionserver-ip-172-20-113-147.log.2020-05-26-17 > 2020-05-26 17:14:50,396 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,397 ERROR > [regionserver/ip-172-20-113-147.us-west-2.compute.internal/172.20.113.147:16020-BucketCacheWriter-0] > bucket.BucketCache: Failed syncing IO engine > 2020-05-26 17:14:50,400 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,401 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: IO errors duration time has exceeded 6ms, disabling > cache, please check your IOEngine > The region server is very busy so it should be constantly getting successful > reads and writes in the bucket cache (it is not as though there was some long > ago error and then no successful IO to clear the error flag). > We are running a busy EMR cluster backed by S3. A bucketcache getting > disabled is a huge performance issue because all reads must go to S3. > Looking at the code, I believe I've found a timing issue. Here is the code > for checking and setting the ioErrorStartTime (taken from BucketCache.java): > > {code:java} > /** >* Check whether we tolerate IO error this time. If the duration of IOEngine >* throwing errors exceeds ioErrorsDurationTimeTolerated, we will disable > the >* cache >*/ > private void checkIOErrorIsTolerated() { > long now = EnvironmentEdgeManager.currentTime(); > if (this.ioErrorStartTime > 0) { > if (cacheEnabled && (now - ioErrorStartTime) > > this.ioErrorsTolerationDuration) { > LOG.error("IO errors duration time has exceeded " + > ioErrorsTolerationDuration + > "ms, disabling cache, please check your IOEngine"); > disableCache(); > } > } else { > this.ioErrorStartTime = now; > } > } > {code} > > And here is the code for clearing the ioErrorStartTime when a successful read > or write is done: > {code:java} > if (this.ioErrorStartTime > 0) { > ioErrorStartTime = -1; > } > {code} > Notice that that if ioErrorStartTime is set to -1 after the first if > statement in checkIOErrorIsTolerated but before (now - ioErrorStartTime), > then (now - ioErrorStartTime) will evaluate to (now + 1) resulting in the > code thinking it has exceeded ioErrorsTolerationDuration. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Comment Edited] (HBASE-24454) BucketCache disabled instantly before error duration toleration is reached due to timing issue
[ https://issues.apache.org/jira/browse/HBASE-24454?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120399#comment-17120399 ] Michael Stack edited comment on HBASE-24454 at 5/31/20, 4:14 AM: - Added you as a contributor [~jacob.leblanc]. Thanks for the patch. was (Author: stack): Added you as a contributor [~jacob.leblanc] > BucketCache disabled instantly before error duration toleration is reached > due to timing issue > -- > > Key: HBASE-24454 > URL: https://issues.apache.org/jira/browse/HBASE-24454 > Project: HBase > Issue Type: Bug > Components: BucketCache >Affects Versions: 1.4.10 > Environment: We saw this in HBase 1.4.10 (EMR 5.28.1), but I believe > the newest code still has this problem. >Reporter: Jacob LeBlanc >Assignee: Jacob LeBlanc >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > We saw an instance where BucketCache was disabled after only two IO error > were encountered at nearly the same time. The following shows all errors from > a region server log for the 2020-05-26 17:00 hour. Notice that there are no > other errors until the 17:14:50 at which time the BucketCache is disabled > because it claims duration time has exceeded 6 ms: > $ grep ERROR hbase-hbase-regionserver-ip-172-20-113-147.log.2020-05-26-17 > 2020-05-26 17:14:50,396 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,397 ERROR > [regionserver/ip-172-20-113-147.us-west-2.compute.internal/172.20.113.147:16020-BucketCacheWriter-0] > bucket.BucketCache: Failed syncing IO engine > 2020-05-26 17:14:50,400 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,401 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: IO errors duration time has exceeded 6ms, disabling > cache, please check your IOEngine > The region server is very busy so it should be constantly getting successful > reads and writes in the bucket cache (it is not as though there was some long > ago error and then no successful IO to clear the error flag). > We are running a busy EMR cluster backed by S3. A bucketcache getting > disabled is a huge performance issue because all reads must go to S3. > Looking at the code, I believe I've found a timing issue. Here is the code > for checking and setting the ioErrorStartTime (taken from BucketCache.java): > > {code:java} > /** >* Check whether we tolerate IO error this time. If the duration of IOEngine >* throwing errors exceeds ioErrorsDurationTimeTolerated, we will disable > the >* cache >*/ > private void checkIOErrorIsTolerated() { > long now = EnvironmentEdgeManager.currentTime(); > if (this.ioErrorStartTime > 0) { > if (cacheEnabled && (now - ioErrorStartTime) > > this.ioErrorsTolerationDuration) { > LOG.error("IO errors duration time has exceeded " + > ioErrorsTolerationDuration + > "ms, disabling cache, please check your IOEngine"); > disableCache(); > } > } else { > this.ioErrorStartTime = now; > } > } > {code} > > And here is the code for clearing the ioErrorStartTime when a successful read > or write is done: > {code:java} > if (this.ioErrorStartTime > 0) { > ioErrorStartTime = -1; > } > {code} > Notice that that if ioErrorStartTime is set to -1 after the first if > statement in checkIOErrorIsTolerated but before (now - ioErrorStartTime), > then (now - ioErrorStartTime) will evaluate to (now + 1) resulting in the > code thinking it has exceeded ioErrorsTolerationDuration. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Resolved] (HBASE-24454) BucketCache disabled instantly before error duration toleration is reached due to timing issue
[ https://issues.apache.org/jira/browse/HBASE-24454?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Stack resolved HBASE-24454. --- Fix Version/s: 2.2.6 1.7.0 2.3.0 3.0.0-alpha-1 Hadoop Flags: Reviewed Assignee: Michael Stack (was: Jacob LeBlanc) Resolution: Fixed Pushed on branch-2.2+ and to branch-1. > BucketCache disabled instantly before error duration toleration is reached > due to timing issue > -- > > Key: HBASE-24454 > URL: https://issues.apache.org/jira/browse/HBASE-24454 > Project: HBase > Issue Type: Bug > Components: BucketCache >Affects Versions: 1.4.10 > Environment: We saw this in HBase 1.4.10 (EMR 5.28.1), but I believe > the newest code still has this problem. >Reporter: Jacob LeBlanc >Assignee: Michael Stack >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > We saw an instance where BucketCache was disabled after only two IO error > were encountered at nearly the same time. The following shows all errors from > a region server log for the 2020-05-26 17:00 hour. Notice that there are no > other errors until the 17:14:50 at which time the BucketCache is disabled > because it claims duration time has exceeded 6 ms: > $ grep ERROR hbase-hbase-regionserver-ip-172-20-113-147.log.2020-05-26-17 > 2020-05-26 17:14:50,396 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,397 ERROR > [regionserver/ip-172-20-113-147.us-west-2.compute.internal/172.20.113.147:16020-BucketCacheWriter-0] > bucket.BucketCache: Failed syncing IO engine > 2020-05-26 17:14:50,400 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: Failed reading block > 9adfad3a603047cfa0c98b16da0b0974_13786 from bucket cache > 2020-05-26 17:14:50,401 ERROR [hfile-prefetch-1589117924884] > bucket.BucketCache: IO errors duration time has exceeded 6ms, disabling > cache, please check your IOEngine > The region server is very busy so it should be constantly getting successful > reads and writes in the bucket cache (it is not as though there was some long > ago error and then no successful IO to clear the error flag). > We are running a busy EMR cluster backed by S3. A bucketcache getting > disabled is a huge performance issue because all reads must go to S3. > Looking at the code, I believe I've found a timing issue. Here is the code > for checking and setting the ioErrorStartTime (taken from BucketCache.java): > > {code:java} > /** >* Check whether we tolerate IO error this time. If the duration of IOEngine >* throwing errors exceeds ioErrorsDurationTimeTolerated, we will disable > the >* cache >*/ > private void checkIOErrorIsTolerated() { > long now = EnvironmentEdgeManager.currentTime(); > if (this.ioErrorStartTime > 0) { > if (cacheEnabled && (now - ioErrorStartTime) > > this.ioErrorsTolerationDuration) { > LOG.error("IO errors duration time has exceeded " + > ioErrorsTolerationDuration + > "ms, disabling cache, please check your IOEngine"); > disableCache(); > } > } else { > this.ioErrorStartTime = now; > } > } > {code} > > And here is the code for clearing the ioErrorStartTime when a successful read > or write is done: > {code:java} > if (this.ioErrorStartTime > 0) { > ioErrorStartTime = -1; > } > {code} > Notice that that if ioErrorStartTime is set to -1 after the first if > statement in checkIOErrorIsTolerated but before (now - ioErrorStartTime), > then (now - ioErrorStartTime) will evaluate to (now + 1) resulting in the > code thinking it has exceeded ioErrorsTolerationDuration. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24436) The store file open and close thread pool should be shared at the region level
[ https://issues.apache.org/jira/browse/HBASE-24436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120397#comment-17120397 ] Zheng Wang commented on HBASE-24436: {quote}What do you think about the solution Andrew provide above, which is creating the thread pool based on store file count with upper bound, which will be faster, right? Any other concerns? {quote} Agree too, would be better. > The store file open and close thread pool should be shared at the region level > -- > > Key: HBASE-24436 > URL: https://issues.apache.org/jira/browse/HBASE-24436 > Project: HBase > Issue Type: Improvement >Reporter: Junhong Xu >Assignee: Junhong Xu >Priority: Minor > > For now, we provide threads per column family evenly in general, but there > are some cases that some column families have much more store files than > others( maybe that's the life, right? ). So in that case, some Stores have > beed done quickly while others are struggling.We should share the thread pool > at the region level in case of data skew. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] saintstack merged pull request #1816: HBASE-24454 - Read ioErrorStartTime to local temporary variable…
saintstack merged pull request #1816: URL: https://github.com/apache/hbase/pull/1816 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-19455) Try to re-enable a disabled BucketCache after a set timeout
[ https://issues.apache.org/jira/browse/HBASE-19455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120396#comment-17120396 ] Anoop Sam John commented on HBASE-19455: +1 for doing this. [~zyork] are u still working on this or else I can take this? > Try to re-enable a disabled BucketCache after a set timeout > --- > > Key: HBASE-19455 > URL: https://issues.apache.org/jira/browse/HBASE-19455 > Project: HBase > Issue Type: Bug >Reporter: Zach York >Assignee: Zach York >Priority: Major > > This JIRA is a follow-up to HBASE-19435. Currently, if the BucketCache is > disabled, the cache will try to enable itself again. This isn’t ideal for > situations where the cache is disabled because of transient issues. Instead, > we should have BucketCache try to re-enable itself after a time period. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] songxincun commented on a change in pull request #1801: [HBASE-24441]CacheConfig details logged at Store open is not really u…
songxincun commented on a change in pull request #1801: URL: https://github.com/apache/hbase/pull/1801#discussion_r432906139 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/CacheConfig.java ## @@ -205,6 +206,7 @@ public CacheConfig(Configuration conf, ColumnFamilyDescriptor family, BlockCache this.blockCache = blockCache; this.byteBuffAllocator = byteBuffAllocator; LOG.info("Created cacheConfig: " + this + (family == null ? "" : " for family " + family) + +(hri == null ? "" : " in region " + hri.getRegionNameAsString()) + " with blockCache=" + blockCache); Review comment: Yes, maybe we can log CacheConfig in the BlockCacheFactory#createBlockCache and remove it from here 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] songxincun commented on pull request #1801: [HBASE-24441]CacheConfig details logged at Store open is not really u…
songxincun commented on pull request #1801: URL: https://github.com/apache/hbase/pull/1801#issuecomment-636415510 Fix what Anoop suggests 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] songxincun commented on a change in pull request #1801: [HBASE-24441]CacheConfig details logged at Store open is not really u…
songxincun commented on a change in pull request #1801: URL: https://github.com/apache/hbase/pull/1801#discussion_r432906034 ## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/CacheConfig.java ## @@ -205,6 +206,7 @@ public CacheConfig(Configuration conf, ColumnFamilyDescriptor family, BlockCache this.blockCache = blockCache; this.byteBuffAllocator = byteBuffAllocator; LOG.info("Created cacheConfig: " + this + (family == null ? "" : " for family " + family) + Review comment: Yes, you are right, thank you for your suggestions 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24436) The store file open and close thread pool should be shared at the region level
[ https://issues.apache.org/jira/browse/HBASE-24436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120394#comment-17120394 ] Anoop Sam John commented on HBASE-24436: bq.which is creating the thread pool based on store file count with upper bound, which will be faster, Ya that would be better. Agree. Looking forward for the patch. How achieving this. Will be bit tricky as we have multi threaded at 2 levels (HStore open which list the store files and then another thread pool for opening store files). > The store file open and close thread pool should be shared at the region level > -- > > Key: HBASE-24436 > URL: https://issues.apache.org/jira/browse/HBASE-24436 > Project: HBase > Issue Type: Improvement >Reporter: Junhong Xu >Assignee: Junhong Xu >Priority: Minor > > For now, we provide threads per column family evenly in general, but there > are some cases that some column families have much more store files than > others( maybe that's the life, right? ). So in that case, some Stores have > beed done quickly while others are struggling.We should share the thread pool > at the region level in case of data skew. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1816: HBASE-24454 - Read ioErrorStartTime to local temporary variable…
Apache-HBase commented on pull request #1816: URL: https://github.com/apache/hbase/pull/1816#issuecomment-636414637 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 5s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 2s | master passed | | +1 :green_heart: | checkstyle | 1m 13s | master passed | | +1 :green_heart: | spotbugs | 2m 9s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 41s | the patch passed | | +1 :green_heart: | checkstyle | 1m 11s | the patch passed | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | hadoopcheck | 12m 13s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 2m 16s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 14s | The patch does not generate ASF License warnings. | | | | 35m 27s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1816 | | Optional Tests | dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle | | uname | Linux f20041178791 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Max. process+thread count | 84 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1816/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] busbey commented on pull request #1813: HBASE-24458 release scripts using docker should specify needed disk consistency
busbey commented on pull request #1813: URL: https://github.com/apache/hbase/pull/1813#issuecomment-636413619 > Perhaps add a quick comment that it is an osx optimization and ignored with other OSes? I didn't do that initially because that's an implementation detail of docker. the consistency statement could be used by docker on other platforms in the future. let me try including something about the OS X optimization on why we're doing it specifically. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Work started] (HBASE-24466) Release scripts in docker mode should be able to use a named volume for maven repo
[ https://issues.apache.org/jira/browse/HBASE-24466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on HBASE-24466 started by Sean Busbey. --- > Release scripts in docker mode should be able to use a named volume for maven > repo > -- > > Key: HBASE-24466 > URL: https://issues.apache.org/jira/browse/HBASE-24466 > Project: HBase > Issue Type: Improvement > Components: build, community >Reporter: Sean Busbey >Assignee: Sean Busbey >Priority: Major > > Allow optional volume name to use to mount the maven repo so we get reuse of > third party artifacts between runs. > Forcefully remove artifacts we build from it (assuming we can't make a > read-only one that's shared for third party dependencies and an isolated one > that's for what we're building) -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] jacob-leblanc opened a new pull request #1816: HBASE-24454 - Read ioErrorStartTime to local temporary variable…
jacob-leblanc opened a new pull request #1816: URL: https://github.com/apache/hbase/pull/1816 …to avoid issue when it is set to -1 between greater than zero check and calculation of error duration 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1801: [HBASE-24441]CacheConfig details logged at Store open is not really u…
Apache-HBase commented on pull request #1801: URL: https://github.com/apache/hbase/pull/1801#issuecomment-636411422 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 21s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 1s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 4s | master passed | | +1 :green_heart: | checkstyle | 1m 14s | master passed | | +1 :green_heart: | spotbugs | 2m 11s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 38s | the patch passed | | +1 :green_heart: | checkstyle | 1m 10s | the patch passed | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | hadoopcheck | 12m 13s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 2m 14s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 13s | The patch does not generate ASF License warnings. | | | | 35m 51s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1801/2/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1801 | | JIRA Issue | HBASE-24441 | | Optional Tests | dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle | | uname | Linux 9381cf33679a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Max. process+thread count | 84 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1801/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1755: HBASE-24069 Provide an ExponentialBackOffPolicy sleep between failed …
Apache-HBase commented on pull request #1755: URL: https://github.com/apache/hbase/pull/1755#issuecomment-636408973 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 6m 58s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | The patch appears to include 1 new or modified test files. | ||| _ branch-1 Compile Tests _ | | +1 :green_heart: | mvninstall | 9m 45s | branch-1 passed | | +1 :green_heart: | compile | 0m 40s | branch-1 passed with JDK v1.8.0_252 | | +1 :green_heart: | compile | 0m 43s | branch-1 passed with JDK v1.7.0_262 | | +1 :green_heart: | checkstyle | 1m 48s | branch-1 passed | | +1 :green_heart: | shadedjars | 3m 1s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 48s | branch-1 passed with JDK v1.8.0_252 | | +1 :green_heart: | javadoc | 0m 42s | branch-1 passed with JDK v1.7.0_262 | | +0 :ok: | spotbugs | 2m 59s | Used deprecated FindBugs config; considering switching to SpotBugs. | | +1 :green_heart: | findbugs | 2m 56s | branch-1 passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 1m 52s | the patch passed | | +1 :green_heart: | compile | 0m 41s | the patch passed with JDK v1.8.0_252 | | +1 :green_heart: | javac | 0m 41s | the patch passed | | +1 :green_heart: | compile | 0m 42s | the patch passed with JDK v1.7.0_262 | | +1 :green_heart: | javac | 0m 42s | the patch passed | | +1 :green_heart: | checkstyle | 1m 37s | hbase-server: The patch generated 0 new + 245 unchanged - 36 fixed = 245 total (was 281) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | shadedjars | 2m 50s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | hadoopcheck | 4m 36s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2. | | +1 :green_heart: | javadoc | 0m 31s | the patch passed with JDK v1.8.0_252 | | +1 :green_heart: | javadoc | 0m 42s | the patch passed with JDK v1.7.0_262 | | +1 :green_heart: | findbugs | 2m 51s | the patch passed | ||| _ Other Tests _ | | -1 :x: | unit | 132m 20s | hbase-server in the patch failed. | | +1 :green_heart: | asflicense | 0m 37s | The patch does not generate ASF License warnings. | | | | 180m 5s | | | Reason | Tests | |---:|:--| | Failed junit tests | hadoop.hbase.procedure.TestFailedProcCleanup | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1755/4/artifact/out/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1755 | | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile | | uname | Linux 6acd71b1f34a 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1755/out/precommit/personality/provided.sh | | git revision | branch-1 / 61fa21e | | Default Java | 1.7.0_262 | | Multi-JDK versions | /usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_262 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1755/4/artifact/out/patch-unit-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1755/4/testReport/ | | Max. process+thread count | 4586 (vs. ulimit of 1) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1755/4/console | | versions | git=1.9.1 maven=3.0.5 findbugs=3.0.1 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1815: HBASE-24477: Move ConfigurationObserver and related classes to hbase-c…
Apache-HBase commented on pull request #1815: URL: https://github.com/apache/hbase/pull/1815#issuecomment-636408896 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m 33s | Docker mode activated. | | -0 :warning: | yetus | 0m 2s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 3m 58s | master passed | | +1 :green_heart: | compile | 1m 15s | master passed | | +1 :green_heart: | shadedjars | 6m 2s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 56s | master passed | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 13s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 46s | the patch passed | | +1 :green_heart: | compile | 1m 17s | the patch passed | | +1 :green_heart: | javac | 1m 17s | the patch passed | | +1 :green_heart: | shadedjars | 6m 2s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 54s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 30s | hbase-common in the patch passed. | | +1 :green_heart: | unit | 198m 12s | hbase-server in the patch passed. | | | | 228m 59s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1815 | | JIRA Issue | HBASE-24477 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 644d7b3f9c3f 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/testReport/ | | Max. process+thread count | 2966 (vs. ulimit of 12500) | | modules | C: hbase-common hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-19663) javadoc creation needs jsr305
[ https://issues.apache.org/jira/browse/HBASE-19663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120386#comment-17120386 ] Hudson commented on HBASE-19663: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > javadoc creation needs jsr305 > - > > Key: HBASE-19663 > URL: https://issues.apache.org/jira/browse/HBASE-19663 > Project: HBase > Issue Type: Bug > Components: documentation, website >Reporter: Michael Stack >Assignee: Sean Busbey >Priority: Major > Fix For: 1.6.0, 1.4.11 > > Attachments: HBASE-19663-branch-1.4.v0.patch, script.sh > > > Cryptic failure trying to build beta-1 RC. Fails like this: > {code} > [INFO] BUILD FAILURE > [INFO] > > [INFO] Total time: 03:54 min > [INFO] Finished at: 2017-12-29T01:13:15-08:00 > [INFO] Final Memory: 381M/9165M > [INFO] > > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-site-plugin:3.4:site (default-site) on project > hbase: Error generating maven-javadoc-plugin:2.10.3:aggregate: > [ERROR] Exit code: 1 - warning: unknown enum constant When.ALWAYS > [ERROR] reason: class file for javax.annotation.meta.When not found > [ERROR] warning: unknown enum constant When.UNKNOWN > [ERROR] warning: unknown enum constant When.MAYBE > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: malformed: "#matchingRows(Cell, byte[]))" > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: reference not found: #matchingRows(Cell, byte[])) > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: reference not found: #matchingRows(Cell, byte[])) > [ERROR] javadoc: warning - Class javax.annotation.Nonnull not found. > [ERROR] javadoc: error - class file for > javax.annotation.meta.TypeQualifierNickname not found > [ERROR] > [ERROR] Command line was: /home/stack/bin/jdk1.8.0_151/jre/../bin/javadoc > -J-Xmx2G @options @packages > [ERROR] > [ERROR] Refer to the generated Javadoc files in > '/home/stack/hbase.git/target/site/apidocs' dir. > [ERROR] -> [Help 1] > [ERROR] > [ERROR] To see the full stack trace of the errors, re-run Maven with the -e > switch. > [ERROR] Re-run Maven using the -X switch to enable full debug logging. > [ERROR] > [ERROR] For more information about the errors and possible solutions, please > read the following articles: > [ERROR] [Help 1] > http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException > {code} > javax.annotation.meta.TypeQualifierNickname is out of jsr305 but we don't > include this anywhere according to mvn dependency. > Happens building the User API both test and main. > Excluding these lines gets us passing again: > {code} > 3511 > 3512 > org.apache.yetus.audience.tools.IncludePublicAnnotationsStandardDoclet > 3513 > 3514 > 3515 org.apache.yetus > 3516 audience-annotations > 3517 ${audience-annotations.version} > 3518 > + 3519 true > {code} > Tried upgrading to newer mvn site (ours is three years old) but that a > different set of problems. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-22033) Update to maven-javadoc-plugin 3.2.0 and switch to non-forking aggregate goals
[ https://issues.apache.org/jira/browse/HBASE-22033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120387#comment-17120387 ] Hudson commented on HBASE-22033: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Update to maven-javadoc-plugin 3.2.0 and switch to non-forking aggregate goals > -- > > Key: HBASE-22033 > URL: https://issues.apache.org/jira/browse/HBASE-22033 > Project: HBase > Issue Type: Task > Components: build, website >Reporter: Sean Busbey >Assignee: Sean Busbey >Priority: Major > Fix For: 3.0.0-alpha-1, 2.4.0 > > > MJAVADOC-444 got into the 3.1.0 release of the maven-javadoc-plugin so now > there are versions of the aggregate javadoc goals that don't include a forked > build. > update our build to make use of this new feature. (a before/after on build > time would be nice to know as well) -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-23716) [hbase-thirdparty] Make release 3.2.0
[ https://issues.apache.org/jira/browse/HBASE-23716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120389#comment-17120389 ] Hudson commented on HBASE-23716: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > [hbase-thirdparty] Make release 3.2.0 > - > > Key: HBASE-23716 > URL: https://issues.apache.org/jira/browse/HBASE-23716 > Project: HBase > Issue Type: Bug > Components: thirdparty >Affects Versions: 3.0.0-alpha-1, 2.3.0 >Reporter: Michael Stack >Assignee: Michael Stack >Priority: Major > Fix For: thirdparty-3.2.0 > > > Update the dependencies in hbase-thirdparty. In particular, pb goes from 3.7 > to 3.11 with a few perf improvements. > Would like to update the hbase-thirdparty to include in the coming > hbase-2.3.0. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24471) The way we bootstrap meta table is confusing
[ https://issues.apache.org/jira/browse/HBASE-24471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120391#comment-17120391 ] Hudson commented on HBASE-24471: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > The way we bootstrap meta table is confusing > > > Key: HBASE-24471 > URL: https://issues.apache.org/jira/browse/HBASE-24471 > Project: HBase > Issue Type: Improvement > Components: master, meta, proc-v2 >Reporter: Duo Zhang >Assignee: Duo Zhang >Priority: Major > Fix For: 3.0.0-alpha-1, 2.4.0 > > > I used to thought the work is only done in the InitMetaProcedure but in fact > we have several pieces of code related to this which are placed in different > classes. > In HRegionServer.initializeFileSystem, we will test whether we are HMaster. > If so, we will call FSTableDescriptors.tryUpdateMetaTableDescriptor. And in > MasterFileSystem.checkRootDir, we will check if meta region is available. If > not, we will bootstrap it, and it will also call > FSTableDescriptors.tryUpdateMetaTableDescriptor. > The problem here is that, HRegionServer.initializeFileSystem is called in the > constructor of HRegionServer, which means it will be executed before > MasterFileSystem.checkRootDir, as we will only create the MasterFileSystem > for active master... > Ideally I prefer we move all the bootstrap code into InitMetaProcedure, which > will be much cleaner. If this can not be done, at least we could move the > code into MasterFileSystem. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-23941) get_slowlog_responses filters with AND/OR operator support
[ https://issues.apache.org/jira/browse/HBASE-23941?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120390#comment-17120390 ] Hudson commented on HBASE-23941: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > get_slowlog_responses filters with AND/OR operator support > -- > > Key: HBASE-23941 > URL: https://issues.apache.org/jira/browse/HBASE-23941 > Project: HBase > Issue Type: Sub-task >Affects Versions: 3.0.0-alpha-1, 2.3.0 >Reporter: Viraj Jasani >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > As of now, get_slowlog_responses support filters for various fields but tries > to match either of multiple filters i.e. default operator is "OR". We can > support user provided "AND" operator so that only entries that match all the > filters can be returned as result. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24343) Document how to configure the http request log
[ https://issues.apache.org/jira/browse/HBASE-24343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120388#comment-17120388 ] Hudson commented on HBASE-24343: Results for branch branch-2 [build #2684 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/2684/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Document how to configure the http request log > -- > > Key: HBASE-24343 > URL: https://issues.apache.org/jira/browse/HBASE-24343 > Project: HBase > Issue Type: Sub-task > Components: documentation >Affects Versions: 2.3.0 >Reporter: Duo Zhang >Assignee: Nick Dimiduk >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1815: HBASE-24477: Move ConfigurationObserver and related classes to hbase-c…
Apache-HBase commented on pull request #1815: URL: https://github.com/apache/hbase/pull/1815#issuecomment-636408596 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m 39s | Docker mode activated. | | -0 :warning: | yetus | 0m 2s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 21s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 4m 41s | master passed | | +1 :green_heart: | compile | 1m 34s | master passed | | +1 :green_heart: | shadedjars | 6m 22s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 18s | hbase-common in master failed. | | -0 :warning: | javadoc | 0m 41s | hbase-server in master failed. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 14s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 4m 34s | the patch passed | | +1 :green_heart: | compile | 1m 33s | the patch passed | | +1 :green_heart: | javac | 1m 33s | the patch passed | | +1 :green_heart: | shadedjars | 6m 20s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 16s | hbase-common in the patch failed. | | -0 :warning: | javadoc | 0m 41s | hbase-server in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 1m 45s | hbase-common in the patch passed. | | +1 :green_heart: | unit | 190m 29s | hbase-server in the patch passed. | | | | 224m 33s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1815 | | JIRA Issue | HBASE-24477 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux d5e6ab063144 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/testReport/ | | Max. process+thread count | 3649 (vs. ulimit of 12500) | | modules | C: hbase-common hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1815: HBASE-24477: Move ConfigurationObserver and related classes to hbase-c…
Apache-HBase commented on pull request #1815: URL: https://github.com/apache/hbase/pull/1815#issuecomment-636394186 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 23s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 3m 40s | master passed | | +1 :green_heart: | checkstyle | 1m 29s | master passed | | +1 :green_heart: | spotbugs | 2m 39s | master passed | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 14s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 22s | the patch passed | | -0 :warning: | checkstyle | 0m 24s | hbase-common: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0) | | +1 :green_heart: | checkstyle | 1m 5s | hbase-server: The patch generated 0 new + 0 unchanged - 4 fixed = 0 total (was 4) | | -0 :warning: | whitespace | 0m 0s | The patch has 4 line(s) that end in whitespace. Use git apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply | | +1 :green_heart: | hadoopcheck | 11m 6s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 2m 59s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 26s | The patch does not generate ASF License warnings. | | | | 35m 47s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1815 | | JIRA Issue | HBASE-24477 | | Optional Tests | dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle | | uname | Linux f96c248f5d44 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt | | whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/artifact/yetus-general-check/output/whitespace-eol.txt | | Max. process+thread count | 94 (vs. ulimit of 12500) | | modules | C: hbase-common hbase-server U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1815/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] bharathv opened a new pull request #1815: HBASE-24477: Move ConfigurationManager and related classes to hbase-c…
bharathv opened a new pull request #1815: URL: https://github.com/apache/hbase/pull/1815 …ommon This utility is useful for any module that wants to detect dynamic config changes. Having it to hbase-common makes it accessible to all the other modules. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Created] (HBASE-24477) Move ConfigurationObserver and related classes to hbase-common
Bharath Vissapragada created HBASE-24477: Summary: Move ConfigurationObserver and related classes to hbase-common Key: HBASE-24477 URL: https://issues.apache.org/jira/browse/HBASE-24477 Project: HBase Issue Type: Task Components: conf Reporter: Bharath Vissapragada Assignee: Bharath Vissapragada It's a nice utility that all the modules can leverage. Putting it in common makes it accessible. I want to use it in client for dynamic reconfiguration of master addresses in HBASE-18095. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] brfrn169 commented on pull request #1648: HBASE-8458 Support for batch version of checkAndMutate()
brfrn169 commented on pull request #1648: URL: https://github.com/apache/hbase/pull/1648#issuecomment-636389446 It looks like the unit tests are okay this time. > Make sure to take a look at the Hadoop3 report and make sure you aren't introducing more javadoc warnings. I glanced at the one and it was the same number of warnings. I checked the report and this patch is not introducing more javadoc warnings. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] bharathv commented on a change in pull request #1755: HBASE-24069 Provide an ExponentialBackOffPolicy sleep between failed …
bharathv commented on a change in pull request #1755: URL: https://github.com/apache/hbase/pull/1755#discussion_r432888949 ## File path: hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestAssignmentManagerOnCluster.java ## @@ -598,6 +605,68 @@ public void testCloseFailed() throws Exception { } } + /** + * This tests region close with exponential backoff + */ + @Test(timeout = 6) + public void testCloseRegionWithExponentialBackOff() throws Exception { +String table = testTableName.getTableName().toString(); Review comment: you already get a "TableName", you don't need to convert it to string and back to TableName. Also, TEST_UTIL.deleteTable() takes TableName as argument. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636369084 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 20s | master passed | | +1 :green_heart: | checkstyle | 0m 13s | master passed | | +1 :green_heart: | spotbugs | 0m 32s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 22s | the patch passed | | -0 :warning: | checkstyle | 0m 14s | hbase-http: The patch generated 7 new + 7 unchanged - 0 fixed = 14 total (was 7) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | xml | 0m 1s | The patch has no ill-formed XML file. | | +1 :green_heart: | hadoopcheck | 11m 4s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 0m 43s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 12s | The patch does not generate ASF License warnings. | | | | 27m 7s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti checkstyle | | uname | Linux 279238ebadbb 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-http.txt | | Max. process+thread count | 94 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636368941 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 8s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 28s | master passed | | +1 :green_heart: | compile | 0m 21s | master passed | | +1 :green_heart: | shadedjars | 6m 23s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 16s | hbase-http in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 32s | the patch passed | | +1 :green_heart: | compile | 0m 23s | the patch passed | | +1 :green_heart: | javac | 0m 23s | the patch passed | | +1 :green_heart: | shadedjars | 6m 25s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 16s | hbase-http in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 47s | hbase-http in the patch passed. | | | | 25m 51s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 7bbf5602824b 4.15.0-91-generic #92-Ubuntu SMP Fri Feb 28 11:09:48 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/testReport/ | | Max. process+thread count | 311 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636368456 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 46s | master passed | | +1 :green_heart: | compile | 0m 20s | master passed | | +1 :green_heart: | shadedjars | 5m 32s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 19s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 22s | the patch passed | | +1 :green_heart: | compile | 0m 21s | the patch passed | | +1 :green_heart: | javac | 0m 21s | the patch passed | | +1 :green_heart: | shadedjars | 5m 30s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 16s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 43s | hbase-http in the patch passed. | | | | 21m 32s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 42becce1 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/testReport/ | | Max. process+thread count | 545 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-20904) Prometheus /metrics http endpoint for monitoring integration
[ https://issues.apache.org/jira/browse/HBASE-20904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120330#comment-17120330 ] Madhusoodan commented on HBASE-20904: - [~busbey] can you please assign this to me > Prometheus /metrics http endpoint for monitoring integration > > > Key: HBASE-20904 > URL: https://issues.apache.org/jira/browse/HBASE-20904 > Project: HBase > Issue Type: New Feature > Components: metrics, monitoring >Reporter: Hari Sekhon >Priority: Major > > Feature Request to add Prometheus /metrics http endpoint for monitoring > integration: > [https://prometheus.io/docs/prometheus/latest/configuration/configuration/#%3Cscrape_config%3E] > Prometheus metrics format for that endpoint: > [https://github.com/prometheus/docs/blob/master/content/docs/instrumenting/exposition_formats.md] > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Comment Edited] (HBASE-20904) Prometheus /metrics http endpoint for monitoring integration
[ https://issues.apache.org/jira/browse/HBASE-20904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120316#comment-17120316 ] Madhusoodan edited comment on HBASE-20904 at 5/30/20, 6:05 PM: --- I worked on this today and submitted a PR (1814). *Implementation / Usage details* /metrics is still supported against Hadoop2 so not using /metrics endpoint. Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note # Most of the servlet code for /prom2 code is cloned from HADOOP-16398 was (Author: mmpataki): I have worked on this today and submitted a PR (1814). *Implementation / Usage details* /metrics is still supported against Hadoop2 so not using /metrics endpoint. Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note # Most of the servlet code for /prom2 code is cloned from HADOOP-16398 > Prometheus /metrics http endpoint for monitoring integration > > > Key: HBASE-20904 > URL: https://issues.apache.org/jira/browse/HBASE-20904 > Project: HBase > Issue Type: New Feature > Components: metrics, monitoring >Reporter: Hari Sekhon >Priority: Major > > Feature Request to add Prometheus /metrics http endpoint for monitoring > integration: > [https://prometheus.io/docs/prometheus/latest/configuration/configuration/#%3Cscrape_config%3E] > Prometheus metrics format for that endpoint: > [https://github.com/prometheus/docs/blob/master/content/docs/instrumenting/exposition_formats.md] > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] mmpataki commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636362249 @busbey I am not understanding this Apache pre-commit builds, why the build fails in first and succeeds in second attempt 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636361543 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 3m 40s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 5m 8s | master passed | | +1 :green_heart: | compile | 0m 23s | master passed | | +1 :green_heart: | shadedjars | 6m 31s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 21s | hbase-http in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 46s | the patch passed | | +1 :green_heart: | compile | 0m 23s | the patch passed | | +1 :green_heart: | javac | 0m 23s | the patch passed | | +1 :green_heart: | shadedjars | 6m 33s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 16s | hbase-http in the patch failed. | ||| _ Other Tests _ | | -1 :x: | unit | 0m 41s | hbase-http in the patch failed. | | | | 29m 36s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux bc42b31f761f 4.15.0-91-generic #92-Ubuntu SMP Fri Feb 28 11:09:48 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/testReport/ | | Max. process+thread count | 280 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636361281 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 43s | master passed | | +1 :green_heart: | checkstyle | 0m 16s | master passed | | +1 :green_heart: | spotbugs | 0m 35s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 20s | the patch passed | | -0 :warning: | checkstyle | 0m 14s | hbase-http: The patch generated 9 new + 7 unchanged - 0 fixed = 16 total (was 7) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | xml | 0m 1s | The patch has no ill-formed XML file. | | +1 :green_heart: | hadoopcheck | 11m 4s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 0m 42s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 14s | The patch does not generate ASF License warnings. | | | | 27m 44s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti checkstyle | | uname | Linux c570bdb20ba9 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-http.txt | | Max. process+thread count | 94 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636361176 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 3m 40s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 17s | master passed | | +1 :green_heart: | compile | 0m 19s | master passed | | +1 :green_heart: | shadedjars | 6m 8s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 17s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 59s | the patch passed | | +1 :green_heart: | compile | 0m 20s | the patch passed | | +1 :green_heart: | javac | 0m 20s | the patch passed | | +1 :green_heart: | shadedjars | 6m 7s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 15s | the patch passed | ||| _ Other Tests _ | | -1 :x: | unit | 0m 40s | hbase-http in the patch failed. | | | | 26m 57s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1814 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux b821b573037d 4.15.0-91-generic #92-Ubuntu SMP Fri Feb 28 11:09:48 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/testReport/ | | Max. process+thread count | 318 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1814/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Comment Edited] (HBASE-20904) Prometheus /metrics http endpoint for monitoring integration
[ https://issues.apache.org/jira/browse/HBASE-20904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120316#comment-17120316 ] Madhusoodan edited comment on HBASE-20904 at 5/30/20, 5:16 PM: --- I have worked on this today and submitted a PR (1814). *Implementation / Usage details* /metrics is still supported against Hadoop2 so not using /metrics endpoint. Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note # Most of the servlet code for /prom2 code is cloned from HADOOP-16398 was (Author: mmpataki): I have worked on this today and submitted a PR (1814). *Implementation* Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note # Most of the servlet code for /prom2 code is cloned from HADOOP-16398 > Prometheus /metrics http endpoint for monitoring integration > > > Key: HBASE-20904 > URL: https://issues.apache.org/jira/browse/HBASE-20904 > Project: HBase > Issue Type: New Feature > Components: metrics, monitoring >Reporter: Hari Sekhon >Priority: Major > > Feature Request to add Prometheus /metrics http endpoint for monitoring > integration: > [https://prometheus.io/docs/prometheus/latest/configuration/configuration/#%3Cscrape_config%3E] > Prometheus metrics format for that endpoint: > [https://github.com/prometheus/docs/blob/master/content/docs/instrumenting/exposition_formats.md] > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-20904) Prometheus /metrics http endpoint for monitoring integration
[ https://issues.apache.org/jira/browse/HBASE-20904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120316#comment-17120316 ] Madhusoodan commented on HBASE-20904: - I have worked on this today and submitted a PR (1814). *Implementation* Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note # Most of the servlet code for /prom2 code is cloned from HADOOP-16398 > Prometheus /metrics http endpoint for monitoring integration > > > Key: HBASE-20904 > URL: https://issues.apache.org/jira/browse/HBASE-20904 > Project: HBase > Issue Type: New Feature > Components: metrics, monitoring >Reporter: Hari Sekhon >Priority: Major > > Feature Request to add Prometheus /metrics http endpoint for monitoring > integration: > [https://prometheus.io/docs/prometheus/latest/configuration/configuration/#%3Cscrape_config%3E] > Prometheus metrics format for that endpoint: > [https://github.com/prometheus/docs/blob/master/content/docs/instrumenting/exposition_formats.md] > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] mmpataki commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636359222 @busbey I have commented on it. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24436) The store file open and close thread pool should be shared at the region level
[ https://issues.apache.org/jira/browse/HBASE-24436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120315#comment-17120315 ] Junhong Xu commented on HBASE-24436: I think we have reached an agreement on creating the thread pool at the region level. The difference is about the parallelism, which is based on stores or store files. [~filtertip] What do you think about the solution Andrew provide above, which is creating the thread pool based on store file count with upper bound, which will be faster, right? Any other concerns? > The store file open and close thread pool should be shared at the region level > -- > > Key: HBASE-24436 > URL: https://issues.apache.org/jira/browse/HBASE-24436 > Project: HBase > Issue Type: Improvement >Reporter: Junhong Xu >Assignee: Junhong Xu >Priority: Minor > > For now, we provide threads per column family evenly in general, but there > are some cases that some column families have much more store files than > others( maybe that's the life, right? ). So in that case, some Stores have > beed done quickly while others are struggling.We should share the thread pool > at the region level in case of data skew. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] busbey commented on pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
busbey commented on pull request #1814: URL: https://github.com/apache/hbase/pull/1814#issuecomment-636358282 Hi! Can you comment on the jira issue so I can assign it to you? 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636357949 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 47s | master passed | | +1 :green_heart: | checkstyle | 0m 16s | master passed | | +1 :green_heart: | spotbugs | 0m 35s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 22s | the patch passed | | -0 :warning: | checkstyle | 0m 14s | hbase-http: The patch generated 9 new + 7 unchanged - 0 fixed = 16 total (was 7) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | xml | 0m 2s | The patch has no ill-formed XML file. | | +1 :green_heart: | hadoopcheck | 11m 4s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 0m 41s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 14s | The patch does not generate ASF License warnings. | | | | 27m 41s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti checkstyle | | uname | Linux a736f2bfe529 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-http.txt | | Max. process+thread count | 94 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636357415 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 17s | master passed | | +1 :green_heart: | compile | 0m 21s | master passed | | +1 :green_heart: | shadedjars | 5m 46s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 20s | hbase-http in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 3s | the patch passed | | +1 :green_heart: | compile | 0m 22s | the patch passed | | +1 :green_heart: | javac | 0m 22s | the patch passed | | +1 :green_heart: | shadedjars | 5m 41s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 17s | hbase-http in the patch failed. | ||| _ Other Tests _ | | -1 :x: | unit | 0m 37s | hbase-http in the patch failed. | | | | 23m 13s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 54a93f0a7176 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/testReport/ | | Max. process+thread count | 487 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636357260 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 34s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 47s | master passed | | +1 :green_heart: | compile | 0m 21s | master passed | | +1 :green_heart: | shadedjars | 5m 42s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 17s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 29s | the patch passed | | +1 :green_heart: | compile | 0m 19s | the patch passed | | +1 :green_heart: | javac | 0m 19s | the patch passed | | +1 :green_heart: | shadedjars | 5m 33s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 15s | the patch passed | ||| _ Other Tests _ | | -1 :x: | unit | 0m 37s | hbase-http in the patch failed. | | | | 21m 51s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 0674d34fee77 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/testReport/ | | Max. process+thread count | 369 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/2/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki opened a new pull request #1814: HBASE-20904 Prometheus /metrics http endpoint for monitoring
mmpataki opened a new pull request #1814: URL: https://github.com/apache/hbase/pull/1814 Since all the metricsources have not moved to new metrics API, I added two servlets. a. /prom (PrometheusServlet) uses the new metrics API. b. /prom2 (PrometheusHadoop2Servlet) uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. Side note 1. Most of the servlet code for /prom2 code is cloned from HADOOP-16398 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki closed pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
mmpataki closed pull request #1812: URL: https://github.com/apache/hbase/pull/1812 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
mmpataki commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636356407 Closing this as I committed on master 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1813: HBASE-24458 release scripts using docker should specify needed disk consistency
Apache-HBase commented on pull request #1813: URL: https://github.com/apache/hbase/pull/1813#issuecomment-636341896 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 28s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 23s | Maven dependency ordering for branch | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 7s | Maven dependency ordering for patch | ||| _ Other Tests _ | | | | 1m 35s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1813 | | Optional Tests | | | uname | Linux 38e4623f891a 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Max. process+thread count | 52 (vs. ulimit of 12500) | | modules | C: U: | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1813: HBASE-24458 release scripts using docker should specify needed disk consistency
Apache-HBase commented on pull request #1813: URL: https://github.com/apache/hbase/pull/1813#issuecomment-636341930 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 30s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 1s | Shelldocs was not available. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for branch | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 7s | Maven dependency ordering for patch | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | ||| _ Other Tests _ | | +0 :ok: | asflicense | 0m 0s | ASF License check generated no output? | | | | 1m 51s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1813 | | Optional Tests | dupname asflicense shellcheck shelldocs | | uname | Linux 62efc9c1651b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Max. process+thread count | 48 (vs. ulimit of 12500) | | modules | C: U: | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) shellcheck=0.4.6 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1813: HBASE-24458 release scripts using docker should specify needed disk consistency
Apache-HBase commented on pull request #1813: URL: https://github.com/apache/hbase/pull/1813#issuecomment-636342016 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 23s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for branch | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 7s | Maven dependency ordering for patch | ||| _ Other Tests _ | | | | 2m 30s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1813 | | Optional Tests | | | uname | Linux 08f3dc2c0f63 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Max. process+thread count | 55 (vs. ulimit of 12500) | | modules | C: U: | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1813/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24343) Document how to configure the http request log
[ https://issues.apache.org/jira/browse/HBASE-24343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120268#comment-17120268 ] Hudson commented on HBASE-24343: Results for branch branch-2.3 [build #113 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/113/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/113/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/113/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/113/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2.3/113/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Document how to configure the http request log > -- > > Key: HBASE-24343 > URL: https://issues.apache.org/jira/browse/HBASE-24343 > Project: HBase > Issue Type: Sub-task > Components: documentation >Affects Versions: 2.3.0 >Reporter: Duo Zhang >Assignee: Nick Dimiduk >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (HBASE-24458) release scripts using docker on mac os should specify needed disk consistency
[ https://issues.apache.org/jira/browse/HBASE-24458?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sean Busbey updated HBASE-24458: Status: Patch Available (was: In Progress) > release scripts using docker on mac os should specify needed disk consistency > - > > Key: HBASE-24458 > URL: https://issues.apache.org/jira/browse/HBASE-24458 > Project: HBase > Issue Type: Improvement >Reporter: Sean Busbey >Assignee: Sean Busbey >Priority: Major > > reading through existing docs and write ups for Docker Desktop (the dominant > docker option for running on OS X machines), it looks like the > host-to-container mapping get significant perf gains if we can relax from > strict container-host consistency on the filesystem. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] busbey opened a new pull request #1813: HBASE-24458 release scripts using docker should specify needed disk consistency
busbey opened a new pull request #1813: URL: https://github.com/apache/hbase/pull/1813 These changes assume that we're running a single release build at at time and that primarily we only need to see stuff outside of the container when it is done. docs on delegated consistency: https://docs.docker.com/storage/bind-mounts/#configure-mount-consistency-for-macos Before this change (using my branch with HBASE-23339 in place to build an RC from master): ``` busbey$ grep -E "(Total time:|uilding binary dist)" output/publish-dist.log + echo '2020-05-16T07:35:10Z Building binary dist' 2020-05-16T07:35:10Z Building binary dist 08:46:50 [INFO] Total time: 01:11 h 11:58:02 [INFO] Total time: 03:11 h 16:33:11 [INFO] Total time: 04:34 h + echo '2020-05-16T16:35:40Z Done building binary distribution' 2020-05-16T16:35:40Z Done building binary distribution ``` and then the final deploy took a bit over 5 hours. after this change we see a pretty dramatic improvement: ``` busbey$ grep -E "(Total time:|uilding binary dist)" output/publish-dist.log + echo '2020-05-29T22:54:08Z Building binary dist' 2020-05-29T22:54:08Z Building binary dist 23:17:49 [INFO] Total time: 23:34 min 23:37:57 [INFO] Total time: 19:56 min 00:49:06 [INFO] Total time: 01:10 h + echo '2020-05-30T00:51:52Z Done building binary distribution' 2020-05-30T00:51:52Z Done building binary distribution busbey$ grep -E "(Total time:|uilding binary dist)" output/mvn_deploy_release.log 03:08:57 [INFO] Total time: 47:13 min ``` 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache-HBase commented on pull request #1811: URL: https://github.com/apache/hbase/pull/1811#issuecomment-636337403 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 50s | Docker mode activated. | | -0 :warning: | yetus | 0m 4s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 37s | master passed | | +1 :green_heart: | compile | 1m 9s | master passed | | +1 :green_heart: | shadedjars | 6m 39s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 46s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 14s | the patch passed | | +1 :green_heart: | compile | 1m 5s | the patch passed | | +1 :green_heart: | javac | 1m 5s | the patch passed | | +1 :green_heart: | shadedjars | 6m 31s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 39s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 224m 59s | hbase-server in the patch passed. | | | | 254m 35s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1811 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 4c6c90cb1b3a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/testReport/ | | Max. process+thread count | 3250 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24428) Priority compaction for recently split daughter regions
[ https://issues.apache.org/jira/browse/HBASE-24428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120257#comment-17120257 ] Hudson commented on HBASE-24428: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Priority compaction for recently split daughter regions > --- > > Key: HBASE-24428 > URL: https://issues.apache.org/jira/browse/HBASE-24428 > Project: HBase > Issue Type: Improvement > Components: Compaction >Reporter: Andrew Kyle Purtell >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.5 > > > We observe that under hotspotting conditions that splitting will proceed very > slowly and the "_Cannot split region due to reference files being there_" log > line will be logged excessively. (branch-1 based production.) This is because > after a region is split it must be compacted before it can be split again. > Reference files must be replaced by real HFiles, normal housekeeping > performed during compaction. However if the regionserver is under excessive > load, its compaction queues may become deep. The daughters of a recently > split hotspotting region may themselves continue to hotspot and will rapidly > need to split again. If the scheduled compaction work to remove/replace > reference files is queued hundreds or thousands of compaction queue elements > behind current, the recently split daughter regions will not be able to split > again for a long time and may grow very large, producing additional > complications (very large regions, very deep replication queues). > To help avoid this condition we should prioritize the compaction of recently > split daughter regions. Compaction requests include a {{priority}} field and > CompactionRequest implements a comparator that sorts by this field. We > already detect when a compaction request involves a region that has reference > files, to ensure that it gets selected to be eligible for compaction, but we > do not seem to prioritize the requests for post-split housekeeping. Split > work should be placed at the top of the queue. Ensure that this is happening. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24343) Document how to configure the http request log
[ https://issues.apache.org/jira/browse/HBASE-24343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120262#comment-17120262 ] Hudson commented on HBASE-24343: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Document how to configure the http request log > -- > > Key: HBASE-24343 > URL: https://issues.apache.org/jira/browse/HBASE-24343 > Project: HBase > Issue Type: Sub-task > Components: documentation >Affects Versions: 2.3.0 >Reporter: Duo Zhang >Assignee: Nick Dimiduk >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24451) Remove the HasThread because the related bug had been fixed since jdk7
[ https://issues.apache.org/jira/browse/HBASE-24451?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120256#comment-17120256 ] Hudson commented on HBASE-24451: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Remove the HasThread because the related bug had been fixed since jdk7 > -- > > Key: HBASE-24451 > URL: https://issues.apache.org/jira/browse/HBASE-24451 > Project: HBase > Issue Type: Improvement > Components: util >Reporter: Zheng Wang >Assignee: Zheng Wang >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > The HasThread was introduced as a wrapper of Thread to workaround a deadlock > bug, and this bug had fixed since jdk7, so I think it is the time to remove > it as we are using jdk8+. > > Related links: > > [https://issues.apache.org/jira/browse/HBASE-4367|https://slack-redir.net/link?url=https%3A%2F%2Fissues.apache.org%2Fjira%2Fbrowse%2FHBASE-4367] > > [https://bugs.java.com/bugdatabase/view_bug.do?bug_id=6915621|https://slack-redir.net/link?url=https%3A%2F%2Fbugs.java.com%2Fbugdatabase%2Fview_bug.do%3Fbug_id%3D6915621] -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-22033) Update to maven-javadoc-plugin 3.2.0 and switch to non-forking aggregate goals
[ https://issues.apache.org/jira/browse/HBASE-22033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120261#comment-17120261 ] Hudson commented on HBASE-22033: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Update to maven-javadoc-plugin 3.2.0 and switch to non-forking aggregate goals > -- > > Key: HBASE-22033 > URL: https://issues.apache.org/jira/browse/HBASE-22033 > Project: HBase > Issue Type: Task > Components: build, website >Reporter: Sean Busbey >Assignee: Sean Busbey >Priority: Major > Fix For: 3.0.0-alpha-1, 2.4.0 > > > MJAVADOC-444 got into the 3.1.0 release of the maven-javadoc-plugin so now > there are versions of the aggregate javadoc goals that don't include a forked > build. > update our build to make use of this new feature. (a before/after on build > time would be nice to know as well) -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-22287) inifinite retries on failed server in RSProcedureDispatcher
[ https://issues.apache.org/jira/browse/HBASE-22287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120259#comment-17120259 ] Hudson commented on HBASE-22287: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > inifinite retries on failed server in RSProcedureDispatcher > --- > > Key: HBASE-22287 > URL: https://issues.apache.org/jira/browse/HBASE-22287 > Project: HBase > Issue Type: Bug >Reporter: Sergey Shelukhin >Assignee: Michael Stack >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > We observed this recently on some cluster, I'm still investigating the root > cause however seems like the retries should have special handling for this > exception; and separately probably a cap on number of retries > {noformat} > 2019-04-20 04:24:27,093 WARN [RSProcedureDispatcher-pool4-t1285] > procedure.RSProcedureDispatcher: request to server ,17020,1555742560432 > failed due to java.io.IOException: Call to :17020 failed on local exception: > org.apache.hadoop.hbase.ipc.FailedServerException: This server is in the > failed servers list: :17020, try=26603, retrying... > {noformat} > The corresponding worker is stuck -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-19663) javadoc creation needs jsr305
[ https://issues.apache.org/jira/browse/HBASE-19663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120260#comment-17120260 ] Hudson commented on HBASE-19663: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > javadoc creation needs jsr305 > - > > Key: HBASE-19663 > URL: https://issues.apache.org/jira/browse/HBASE-19663 > Project: HBase > Issue Type: Bug > Components: documentation, website >Reporter: Michael Stack >Assignee: Sean Busbey >Priority: Major > Fix For: 1.6.0, 1.4.11 > > Attachments: HBASE-19663-branch-1.4.v0.patch, script.sh > > > Cryptic failure trying to build beta-1 RC. Fails like this: > {code} > [INFO] BUILD FAILURE > [INFO] > > [INFO] Total time: 03:54 min > [INFO] Finished at: 2017-12-29T01:13:15-08:00 > [INFO] Final Memory: 381M/9165M > [INFO] > > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-site-plugin:3.4:site (default-site) on project > hbase: Error generating maven-javadoc-plugin:2.10.3:aggregate: > [ERROR] Exit code: 1 - warning: unknown enum constant When.ALWAYS > [ERROR] reason: class file for javax.annotation.meta.When not found > [ERROR] warning: unknown enum constant When.UNKNOWN > [ERROR] warning: unknown enum constant When.MAYBE > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: malformed: "#matchingRows(Cell, byte[]))" > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: reference not found: #matchingRows(Cell, byte[])) > [ERROR] > /home/stack/hbase.git/hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java:762: > warning - Tag @link: reference not found: #matchingRows(Cell, byte[])) > [ERROR] javadoc: warning - Class javax.annotation.Nonnull not found. > [ERROR] javadoc: error - class file for > javax.annotation.meta.TypeQualifierNickname not found > [ERROR] > [ERROR] Command line was: /home/stack/bin/jdk1.8.0_151/jre/../bin/javadoc > -J-Xmx2G @options @packages > [ERROR] > [ERROR] Refer to the generated Javadoc files in > '/home/stack/hbase.git/target/site/apidocs' dir. > [ERROR] -> [Help 1] > [ERROR] > [ERROR] To see the full stack trace of the errors, re-run Maven with the -e > switch. > [ERROR] Re-run Maven using the -X switch to enable full debug logging. > [ERROR] > [ERROR] For more information about the errors and possible solutions, please > read the following articles: > [ERROR] [Help 1] > http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException > {code} > javax.annotation.meta.TypeQualifierNickname is out of jsr305 but we don't > include this anywhere according to mvn dependency. > Happens building the User API both test and main. > Excluding these lines gets us passing again: > {code} > 3511 > 3512 > org.apache.yetus.audience.tools.IncludePublicAnnotationsStandardDoclet > 3513 > 3514 > 3515 org.apache.yetus > 3516 audience-annotations > 3517 ${audience-annotations.version} > 3518 > + 3519 true > {code} > Tried upgrading to newer mvn site (ours is three years old) but that a > different set of problems. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24470) Add store file info when log meessages in HStore.
[ https://issues.apache.org/jira/browse/HBASE-24470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120258#comment-17120258 ] Hudson commented on HBASE-24470: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Add store file info when log meessages in HStore. > - > > Key: HBASE-24470 > URL: https://issues.apache.org/jira/browse/HBASE-24470 > Project: HBase > Issue Type: Improvement > Components: logging, regionserver >Affects Versions: 3.0.0-alpha-1 >Reporter: song XinCun >Assignee: song XinCun >Priority: Minor > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > Some log message do not have store file info when log, need to add it. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-24471) The way we bootstrap meta table is confusing
[ https://issues.apache.org/jira/browse/HBASE-24471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120264#comment-17120264 ] Hudson commented on HBASE-24471: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > The way we bootstrap meta table is confusing > > > Key: HBASE-24471 > URL: https://issues.apache.org/jira/browse/HBASE-24471 > Project: HBase > Issue Type: Improvement > Components: master, meta, proc-v2 >Reporter: Duo Zhang >Assignee: Duo Zhang >Priority: Major > Fix For: 3.0.0-alpha-1, 2.4.0 > > > I used to thought the work is only done in the InitMetaProcedure but in fact > we have several pieces of code related to this which are placed in different > classes. > In HRegionServer.initializeFileSystem, we will test whether we are HMaster. > If so, we will call FSTableDescriptors.tryUpdateMetaTableDescriptor. And in > MasterFileSystem.checkRootDir, we will check if meta region is available. If > not, we will bootstrap it, and it will also call > FSTableDescriptors.tryUpdateMetaTableDescriptor. > The problem here is that, HRegionServer.initializeFileSystem is called in the > constructor of HRegionServer, which means it will be executed before > MasterFileSystem.checkRootDir, as we will only create the MasterFileSystem > for active master... > Ideally I prefer we move all the bootstrap code into InitMetaProcedure, which > will be much cleaner. If this can not be done, at least we could move the > code into MasterFileSystem. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (HBASE-23941) get_slowlog_responses filters with AND/OR operator support
[ https://issues.apache.org/jira/browse/HBASE-23941?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120263#comment-17120263 ] Hudson commented on HBASE-23941: Results for branch master [build #1741 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/]: (/) *{color:green}+1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/General_20Nightly_20Build_20Report/] (/) {color:green}+1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1698/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (/) {color:green}+1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://builds.apache.org/job/HBase%20Nightly/job/master/1741/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > get_slowlog_responses filters with AND/OR operator support > -- > > Key: HBASE-23941 > URL: https://issues.apache.org/jira/browse/HBASE-23941 > Project: HBase > Issue Type: Sub-task >Affects Versions: 3.0.0-alpha-1, 2.3.0 >Reporter: Viraj Jasani >Assignee: Viraj Jasani >Priority: Major > Fix For: 3.0.0-alpha-1, 2.3.0 > > > As of now, get_slowlog_responses support filters for various fields but tries > to match either of multiple filters i.e. default operator is "OR". We can > support user provided "AND" operator so that only entries that match all the > filters can be returned as result. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Created] (HBASE-24476) release scripts should provide timing information
Sean Busbey created HBASE-24476: --- Summary: release scripts should provide timing information Key: HBASE-24476 URL: https://issues.apache.org/jira/browse/HBASE-24476 Project: HBase Issue Type: Improvement Reporter: Sean Busbey Assignee: Sean Busbey right now I can get timing from the individual maven commands but it would be nice to get higher level times. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache-HBase commented on pull request #1811: URL: https://github.com/apache/hbase/pull/1811#issuecomment-636325635 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 31s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 9s | master passed | | +1 :green_heart: | compile | 1m 4s | master passed | | +1 :green_heart: | shadedjars | 6m 2s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 44s | hbase-server in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 4s | the patch passed | | +1 :green_heart: | compile | 1m 4s | the patch passed | | +1 :green_heart: | javac | 1m 4s | the patch passed | | +1 :green_heart: | shadedjars | 5m 52s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 39s | hbase-server in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 127m 37s | hbase-server in the patch passed. | | | | 154m 1s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1811 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 2a2876b3a4e8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/testReport/ | | Max. process+thread count | 4498 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1810: HBASE-24475 Clean up the master thread name getting in SplitLogManage…
Apache-HBase commented on pull request #1810: URL: https://github.com/apache/hbase/pull/1810#issuecomment-636321889 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 2m 36s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 46s | master passed | | +1 :green_heart: | compile | 1m 9s | master passed | | +1 :green_heart: | shadedjars | 6m 24s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 46s | hbase-server in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 5m 20s | the patch passed | | +1 :green_heart: | compile | 1m 25s | the patch passed | | +1 :green_heart: | javac | 1m 25s | the patch passed | | +1 :green_heart: | shadedjars | 7m 8s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 50s | hbase-server in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 189m 33s | hbase-server in the patch passed. | | | | 221m 43s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1810 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 33500b42c133 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/testReport/ | | Max. process+thread count | 3387 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1810: HBASE-24475 Clean up the master thread name getting in SplitLogManage…
Apache-HBase commented on pull request #1810: URL: https://github.com/apache/hbase/pull/1810#issuecomment-636321706 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 28s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 22s | master passed | | +1 :green_heart: | compile | 1m 0s | master passed | | +1 :green_heart: | shadedjars | 6m 2s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 39s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 48s | the patch passed | | +1 :green_heart: | compile | 1m 0s | the patch passed | | +1 :green_heart: | javac | 1m 0s | the patch passed | | +1 :green_heart: | shadedjars | 6m 7s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 36s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 194m 43s | hbase-server in the patch passed. | | | | 220m 31s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1810 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux b07022c92976 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/testReport/ | | Max. process+thread count | 3776 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1810/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636321387 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 32s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 39s | master passed | | +1 :green_heart: | checkstyle | 0m 15s | master passed | | +1 :green_heart: | spotbugs | 0m 34s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 20s | the patch passed | | -0 :warning: | checkstyle | 0m 14s | hbase-http: The patch generated 6 new + 7 unchanged - 0 fixed = 13 total (was 7) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | xml | 0m 1s | The patch has no ill-formed XML file. | | +1 :green_heart: | hadoopcheck | 11m 6s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 0m 42s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 14s | The patch does not generate ASF License warnings. | | | | 27m 45s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti checkstyle | | uname | Linux 25c110132d40 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-http.txt | | Max. process+thread count | 94 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636321365 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 38s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 5m 0s | master passed | | +1 :green_heart: | compile | 0m 23s | master passed | | +1 :green_heart: | shadedjars | 6m 37s | branch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 20s | hbase-http in master failed. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 43s | the patch passed | | +1 :green_heart: | compile | 0m 24s | the patch passed | | +1 :green_heart: | javac | 0m 24s | the patch passed | | +1 :green_heart: | shadedjars | 6m 30s | patch has no errors when building our shaded downstream artifacts. | | -0 :warning: | javadoc | 0m 16s | hbase-http in the patch failed. | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 47s | hbase-http in the patch passed. | | | | 27m 30s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux b8ed1be7a338 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 2020-01-14 | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt | | javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/testReport/ | | Max. process+thread count | 376 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
Apache-HBase commented on pull request #1812: URL: https://github.com/apache/hbase/pull/1812#issuecomment-636321087 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 39s | Docker mode activated. | | -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 13s | master passed | | +1 :green_heart: | compile | 0m 20s | master passed | | +1 :green_heart: | shadedjars | 6m 12s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 17s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 57s | the patch passed | | +1 :green_heart: | compile | 0m 21s | the patch passed | | +1 :green_heart: | javac | 0m 21s | the patch passed | | +1 :green_heart: | shadedjars | 6m 9s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 15s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 46s | hbase-http in the patch passed. | | | | 25m 3s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1812 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 5839accda048 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | Default Java | 1.8.0_232 | | Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/testReport/ | | Max. process+thread count | 405 (vs. ulimit of 12500) | | modules | C: hbase-http U: hbase-http | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1812/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] mmpataki opened a new pull request #1812: HBASE-20904 : Prometheus /metrics http endpoint for monitoring
mmpataki opened a new pull request #1812: URL: https://github.com/apache/hbase/pull/1812 Since all the metricsources have not moved to new metrics API, I added two servlets. a. PrometheusServlet uses the new metrics API. b. PrometheusHadoop2Servlet uses the old hadoop2 metrics API PrometheusHadoop2Servlet can be thrown out once all the metricsources start using the new API. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-24470) Add store file info when log meessages in HStore.
[ https://issues.apache.org/jira/browse/HBASE-24470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120209#comment-17120209 ] Hudson commented on HBASE-24470: Results for branch branch-1 [build #1303 on builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/1303/]: (x) *{color:red}-1 overall{color}* details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/1303//General_Nightly_Build_Report/] (x) {color:red}-1 jdk7 checks{color} -- For more information [see jdk7 report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/1303//JDK7_Nightly_Build_Report/] (x) {color:red}-1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/1303//JDK8_Nightly_Build_Report_(Hadoop2)/] (x) {color:red}-1 source release artifact{color} -- See build output for details. > Add store file info when log meessages in HStore. > - > > Key: HBASE-24470 > URL: https://issues.apache.org/jira/browse/HBASE-24470 > Project: HBase > Issue Type: Improvement > Components: logging, regionserver >Affects Versions: 3.0.0-alpha-1 >Reporter: song XinCun >Assignee: song XinCun >Priority: Minor > Fix For: 3.0.0-alpha-1, 2.3.0, 1.7.0, 2.2.6 > > > Some log message do not have store file info when log, need to add it. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache-HBase commented on pull request #1811: URL: https://github.com/apache/hbase/pull/1811#issuecomment-636312796 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 1m 19s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 4m 7s | master passed | | +1 :green_heart: | checkstyle | 1m 23s | master passed | | +1 :green_heart: | spotbugs | 2m 15s | master passed | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 55s | the patch passed | | -0 :warning: | checkstyle | 1m 17s | hbase-server: The patch generated 4 new + 291 unchanged - 0 fixed = 295 total (was 291) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | hadoopcheck | 12m 30s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. | | +1 :green_heart: | spotbugs | 2m 30s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 14s | The patch does not generate ASF License warnings. | | | | 37m 30s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1811 | | Optional Tests | dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle | | uname | Linux 224d1c16da7a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt | | Max. process+thread count | 84 (vs. ulimit of 12500) | | modules | C: hbase-server U: hbase-server | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1811/1/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache-HBase commented on pull request #1799: HBASE-24455 Correct the doc of "On the number of column families"
Apache-HBase commented on pull request #1799: URL: https://github.com/apache/hbase/pull/1799#issuecomment-636312012 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |::|--:|:|:| | +0 :ok: | reexec | 0m 29s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | ||| _ master Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 47s | master passed | | +0 :ok: | refguide | 4m 50s | branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 3m 22s | the patch passed | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +0 :ok: | refguide | 6m 4s | patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect. | ||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 19s | The patch does not generate ASF License warnings. | | | | 20m 7s | | | Subsystem | Report/Notes | |--:|:-| | Docker | Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1799/3/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/1799 | | Optional Tests | dupname asflicense refguide | | uname | Linux 13053b3581f1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 36a6ef9cf9 | | refguide | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1799/3/artifact/yetus-general-check/output/branch-site/book.html | | refguide | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1799/3/artifact/yetus-general-check/output/patch-site/book.html | | Max. process+thread count | 79 (vs. ulimit of 12500) | | modules | C: . U: . | | Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1799/3/console | | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) | | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org | This message was automatically generated. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[jira] [Commented] (HBASE-23340) hmaster /hbase/replication/rs session expired (hbase replication default value is true, we don't use ) causes logcleaner can not clean oldWALs, which resulits in old
[ https://issues.apache.org/jira/browse/HBASE-23340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17120201#comment-17120201 ] Anoop Sam John commented on HBASE-23340: Pls check HBASE-15234. Seems that solves the issue. This is fixed in 1.2.2. Pls backport or upgrade ur version and check. We can close this issue then. > hmaster /hbase/replication/rs session expired (hbase replication default > value is true, we don't use ) causes logcleaner can not clean oldWALs, which > resulits in oldWALs too large (more than 2TB) > - > > Key: HBASE-23340 > URL: https://issues.apache.org/jira/browse/HBASE-23340 > Project: HBase > Issue Type: Improvement > Components: master >Reporter: jackylau >Assignee: jackylau >Priority: Major > Fix For: master > > Attachments: Snipaste_2019-11-21_10-39-25.png, > Snipaste_2019-11-21_14-10-36.png > > > hmaster /hbase/replication/rs session expired (hbase replication default > value is true, we don't use ) causes logcleaner can not clean oldWALs, which > resulits in oldWALs too large (more than 2TB). > !Snipaste_2019-11-21_10-39-25.png! > > !Snipaste_2019-11-21_14-10-36.png! > > we can solve it by following : > 1) increase the session timeout(but i think it is not a good idea. because we > do not know how long to set is suitable) > 2) close the hbase replication. It is not a good idea too, when our user uses > this feature > 3) we need add retry times, for example when it has already happened three > times, we set the ReplicationLogCleaner and SnapShotCleaner stop > that is all my ideas, i do not konw it is suitable, If it is suitable, could > i commit a PR? > Does anynode have a good idea. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [hbase] Apache-HBase commented on pull request #1799: HBASE-24455 Correct the doc of "On the number of column families"
Apache-HBase commented on pull request #1799: URL: https://github.com/apache/hbase/pull/1799#issuecomment-636310200 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache9 commented on pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache9 commented on pull request #1811: URL: https://github.com/apache/hbase/pull/1811#issuecomment-636308668 The new useMetaCellComparator parameter is for storing root table in the future. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] bsglz commented on pull request #1799: HBASE-24455 Correct the doc of "On the number of column families"
bsglz commented on pull request #1799: URL: https://github.com/apache/hbase/pull/1799#issuecomment-636308633 Ok, will fix later. Thanks. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache9 commented on pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache9 commented on pull request #1811: URL: https://github.com/apache/hbase/pull/1811#issuecomment-636308428 @ndimiduk PTAL. Thanks. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache9 opened a new pull request #1811: HBASE-24474 Rename LocalRegion to MasterRegion
Apache9 opened a new pull request #1811: URL: https://github.com/apache/hbase/pull/1811 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] wchevreuil commented on pull request #1799: HBASE-24455 Correct the doc of "On the number of column families"
wchevreuil commented on pull request #1799: URL: https://github.com/apache/hbase/pull/1799#issuecomment-636308123 > Sounds reasonable, WDYT changed to this: > _HBase currently does not do well with anything above two or three column families so keep the number of column families in your schema low. Currently, flushing is done on a per Region basis so if one column family is carrying the bulk of the data bringing on flushes, the adjacent families will also be flushed even though the amount of data they carry is small. When many column families exist the flushing interaction can make for a bunch of needless i/o (To be addressed by changing flushing to work on a per column family basis). **In addition, compactions triggered at table/region level will happen per store too.**_ +1, LGTM 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] Apache9 commented on a change in pull request #1802: HBASE-24444 Should shutdown mini cluster after class in TestMetaAssignmentWithStopMaster
Apache9 commented on a change in pull request #1802: URL: https://github.com/apache/hbase/pull/1802#discussion_r432826147 ## File path: hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaAssignmentWithStopMaster.java ## @@ -57,6 +58,15 @@ public static void setUp() throws Exception { UTIL.startMiniCluster(option); } + @AfterClass + public static void cleanup() { +try { + UTIL.shutdownMiniCluster(); +} catch (Exception e) { Review comment: Just throw the exception out? 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [hbase] infraio commented on pull request #1805: HBASE-24467 Backport HBASE-23963: Split TestFromClientSide; it takes …
infraio commented on pull request #1805: URL: https://github.com/apache/hbase/pull/1805#issuecomment-636303605 The failed ut related. Let me check it. 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 the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org