[jira] [Commented] (HBASE-27301) Add Delete addFamilyVersion timestamp verify

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580047#comment-17580047
 ] 

Hudson commented on HBASE-27301:


Results for branch branch-2
[build #618 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/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 Delete addFamilyVersion  timestamp verify
> -
>
> Key: HBASE-27301
> URL: https://issues.apache.org/jira/browse/HBASE-27301
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client
>Reporter: zhengsicheng
>Assignee: zhengsicheng
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27299) Bump minimum hadoop 2 version to 2.10.2

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27299?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580049#comment-17580049
 ] 

Hudson commented on HBASE-27299:


Results for branch branch-2
[build #618 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/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}


> Bump minimum hadoop 2 version to 2.10.2
> ---
>
> Key: HBASE-27299
> URL: https://issues.apache.org/jira/browse/HBASE-27299
> Project: HBase
>  Issue Type: Task
>  Components: hadoop2, security
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0
>
>
> Although 2.10.2 still have some security issues, at least it is better than 
> other hadoop2 releases and we still want to maintain hadoop 2 support for 
> branch-2.x.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27296) Some Cell's implementation of toString() such as IndividualBytesFieldCell prints out value and tags which is too verbose

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580050#comment-17580050
 ] 

Hudson commented on HBASE-27296:


Results for branch branch-2
[build #618 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/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}


> Some Cell's implementation of toString() such as IndividualBytesFieldCell 
> prints out value and tags which is too verbose
> 
>
> Key: HBASE-27296
> URL: https://issues.apache.org/jira/browse/HBASE-27296
> Project: HBase
>  Issue Type: Improvement
>  Components: logging
>Affects Versions: 2.4.12
>Reporter: Huaxiang Sun
>Assignee: Huaxiang Sun
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> One of users sees cells >10Mb are logged when over limit at their client log. 
> Checked the code, toString() behavior is not consistent, mostly does not 
> include values and tags. Change toString() to exclude tags/value.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-26982) Add index and bloom filter statistics of LruBlockCache on rs web UI

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-26982?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580051#comment-17580051
 ] 

Hudson commented on HBASE-26982:


Results for branch branch-2
[build #618 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/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 index and bloom filter statistics of LruBlockCache on rs web UI
> ---
>
> Key: HBASE-26982
> URL: https://issues.apache.org/jira/browse/HBASE-26982
> Project: HBase
>  Issue Type: Improvement
>  Components: BlockCache, UI
>Reporter: Xuesen Liang
>Assignee: Xuesen Liang
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4
>
> Attachments: 165520264-e172911b-58c5-4e74-8d61-00bd66524c94.png
>
>
> When _CombinedBlockCache_ is configured, _LruBlockCache_ is used for 
> index/bloom/meta block cache, and _BucketCache_ is used for data block cache. 
> Index and bloom filter statistics on rs web UI will be helpful.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27221) Bump spotless version to 2.24.1

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580048#comment-17580048
 ] 

Hudson commented on HBASE-27221:


Results for branch branch-2
[build #618 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/618/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}


> Bump spotless version to 2.24.1
> ---
>
> Key: HBASE-27221
> URL: https://issues.apache.org/jira/browse/HBASE-27221
> Project: HBase
>  Issue Type: Task
>  Components: build, pom
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> To catch up with the lastest version.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27296) Some Cell's implementation of toString() such as IndividualBytesFieldCell prints out value and tags which is too verbose

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580041#comment-17580041
 ] 

Hudson commented on HBASE-27296:


Results for branch branch-2.4
[build #410 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/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}


> Some Cell's implementation of toString() such as IndividualBytesFieldCell 
> prints out value and tags which is too verbose
> 
>
> Key: HBASE-27296
> URL: https://issues.apache.org/jira/browse/HBASE-27296
> Project: HBase
>  Issue Type: Improvement
>  Components: logging
>Affects Versions: 2.4.12
>Reporter: Huaxiang Sun
>Assignee: Huaxiang Sun
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> One of users sees cells >10Mb are logged when over limit at their client log. 
> Checked the code, toString() behavior is not consistent, mostly does not 
> include values and tags. Change toString() to exclude tags/value.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27221) Bump spotless version to 2.24.1

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580040#comment-17580040
 ] 

Hudson commented on HBASE-27221:


Results for branch branch-2.4
[build #410 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/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}


> Bump spotless version to 2.24.1
> ---
>
> Key: HBASE-27221
> URL: https://issues.apache.org/jira/browse/HBASE-27221
> Project: HBase
>  Issue Type: Task
>  Components: build, pom
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> To catch up with the lastest version.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27301) Add Delete addFamilyVersion timestamp verify

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580039#comment-17580039
 ] 

Hudson commented on HBASE-27301:


Results for branch branch-2.4
[build #410 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.4/410/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 Delete addFamilyVersion  timestamp verify
> -
>
> Key: HBASE-27301
> URL: https://issues.apache.org/jira/browse/HBASE-27301
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client
>Reporter: zhengsicheng
>Assignee: zhengsicheng
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache9 commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache9 commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1216128668

   @ndimiduk FYI.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on a diff in pull request #4691: HBASE-20904 Prometheus /metrics http endpoint for monitoring

2022-08-15 Thread GitBox


Apache9 commented on code in PR #4691:
URL: https://github.com/apache/hbase/pull/4691#discussion_r946323840


##
hbase-http/src/main/java/org/apache/hadoop/hbase/http/prometheus/PrometheusHadoopServlet.java:
##
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.hadoop.hbase.http.prometheus;
+
+import com.google.errorprone.annotations.RestrictedApi;
+import java.io.IOException;
+import java.io.Writer;
+import java.util.Collection;
+import java.util.regex.Pattern;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import org.apache.commons.lang3.StringUtils;
+import org.apache.hadoop.metrics2.AbstractMetric;
+import org.apache.hadoop.metrics2.MetricType;
+import org.apache.hadoop.metrics2.MetricsRecord;
+import org.apache.hadoop.metrics2.MetricsTag;
+import org.apache.hadoop.metrics2.impl.MetricsExportHelper;
+import org.apache.yetus.audience.InterfaceAudience;
+
+@InterfaceAudience.Private
+public class PrometheusHadoopServlet extends HttpServlet {
+
+  private static final Pattern SPLIT_PATTERN =
+Pattern.compile("(?<=[a-z])(?=[A-Z])|(?<=[A-Z])(?=([A-Z][a-z]))|\\W|(_)+");
+
+  @Override
+  protected void doGet(HttpServletRequest req, HttpServletResponse resp) 
throws IOException {
+writeMetrics(resp.getWriter());
+  }
+
+  static String toPrometheusName(String metricRecordName, String metricName) {
+String baseName = metricRecordName + StringUtils.capitalize(metricName);
+String[] parts = SPLIT_PATTERN.split(baseName);
+return String.join("_", parts).toLowerCase();
+  }
+
+  /*
+   * SimpleClient for Prometheus is not used, because the format is very easy 
to implement and this
+   * solution doesn't add any dependencies to the project. You can check the 
Prometheus format here:
+   * https://prometheus.io/docs/instrumenting/exposition_formats/
+   */
+  @RestrictedApi(explanation = "Should only be called in tests", link = "",

Review Comment:
   We also call it in this class so we need to change the allowedOnPath to also 
include the current file.



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27213) Add support for claim queue operation

2022-08-15 Thread Duo Zhang (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17580034#comment-17580034
 ] 

Duo Zhang commented on HBASE-27213:
---

After reconsideration, we have peer lock when executing peer related 
procedures, so it is impossible that during the SCP execution, we remove and 
add the same peer again, as when removing peer we will wait until all ongoing 
SCPs to finish.

But anyway, making peer queue identical is still worth to do, but not a blocker 
for this issue.

> Add support for claim queue operation
> -
>
> Key: HBASE-27213
> URL: https://issues.apache.org/jira/browse/HBASE-27213
> Project: HBase
>  Issue Type: Sub-task
>  Components: Replication
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache9 commented on a diff in pull request #4707: HBASE-27303 Unnecessary replication to secondary region replicas shou…

2022-08-15 Thread GitBox


Apache9 commented on code in PR #4707:
URL: https://github.com/apache/hbase/pull/4707#discussion_r946311009


##
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java:
##
@@ -8040,11 +8040,17 @@ private WriteEntry doWALAppend(WALEdit walEdit, 
BatchOperation batchOp,
 try {
   long txid = this.wal.appendData(this.getRegionInfo(), walKey, walEdit);
   WriteEntry writeEntry = walKey.getWriteEntry();
-  this.attachRegionReplicationInWALAppend(batchOp, miniBatchOp, walKey, 
walEdit, writeEntry);
   // Call sync on our edit.
   if (txid != 0) {
 sync(txid, batchOp.durability);
   }
+  /**
+   * If above {@link HRegion#sync} throws Exception, the RegionServer 
should be aborted and
+   * following {@link BatchOperation#writeMiniBatchOperationsToMemStore} 
will not be executed,
+   * so there is no need to replicate to secondary replica, for this 
reason here we attach the
+   * region replication action after the {@link HRegion#sync} is 
successful.
+   */
+  this.attachRegionReplicationInWALAppend(batchOp, miniBatchOp, walKey, 
walEdit, writeEntry);

Review Comment:
   For most cases the sync will succeeded, attach it before WAL sync can speed 
up the replication? No?



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4706: Backport "HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format" to branch-2

2022-08-15 Thread GitBox


Apache9 commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1216104129

   > I mean, maybe @BukrosSzabolcs or @busbey can confirm. I'd rather not 
backport changes that are not required.
   
   The problem on master is we change to use slf4j style formatter but actually 
we do not use slf4j here, it is String.format...
   
   On branch-2 we even do not use String.format, we just use '+' to concat the 
error message so no actually bug...


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4657: HBASE-27246 RSGroupMappingScript#getRSGroup Should be Synchronized

2022-08-15 Thread GitBox


Apache9 commented on PR #4657:
URL: https://github.com/apache/hbase/pull/4657#issuecomment-1216052056

   Let's change the title?


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4705: HBASE-27279 Make SslHandler work with SaslWrapHandler/SaslUnwrapHandler

2022-08-15 Thread GitBox


Apache9 commented on PR #4705:
URL: https://github.com/apache/hbase/pull/4705#issuecomment-1216044021

   > +1 Although I don't understand every single bit of this change, it looks 
good to me. I believe the key is to replace `addFirst()` calls by something 
that refers to tha actual pipeline.
   
   Yes, we need preserve the original order of the handler other than 
SslHandler while keep SslHandler at the first place.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27221) Bump spotless version to 2.24.1

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579975#comment-17579975
 ] 

Hudson commented on HBASE-27221:


Results for branch master
[build #658 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/General_20Nightly_20Build_20Report/]




(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/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}


> Bump spotless version to 2.24.1
> ---
>
> Key: HBASE-27221
> URL: https://issues.apache.org/jira/browse/HBASE-27221
> Project: HBase
>  Issue Type: Task
>  Components: build, pom
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> To catch up with the lastest version.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27301) Add Delete addFamilyVersion timestamp verify

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579974#comment-17579974
 ] 

Hudson commented on HBASE-27301:


Results for branch master
[build #658 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/]: 
(/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/General_20Nightly_20Build_20Report/]




(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/master/658/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 Delete addFamilyVersion  timestamp verify
> -
>
> Key: HBASE-27301
> URL: https://issues.apache.org/jira/browse/HBASE-27301
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client
>Reporter: zhengsicheng
>Assignee: zhengsicheng
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4698: HBASE-27238 backport backup restore to 2.x

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4698:
URL: https://github.com/apache/hbase/pull/4698#issuecomment-1215909136

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 25s |  Docker mode activated.  |
   | -0 :warning: |  yetus  |   0m  5s |  Unprocessed flag(s): 
--brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list 
--whitespace-tabs-ignore-list --quick-hadoopcheck  |
   ||| _ Prechecks _ |
   ||| _ branch-2.5 Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 17s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 24s |  branch-2.5 passed  |
   | +1 :green_heart: |  compile  |   1m 35s |  branch-2.5 passed  |
   | +1 :green_heart: |  shadedjars  |   3m 42s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 33s |  branch-2.5 passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 16s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 12s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 34s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 34s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 45s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 48s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  | 279m  6s |  root in the patch failed.  |
   |  |   | 304m  5s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4698 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 9d6f65f817c2 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 
20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | branch-2.5 / 7f6f184f06 |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | unit | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt
 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/testReport/
 |
   | Max. process+thread count | 2137 (vs. ulimit of 3) |
   | modules | C: hbase-protocol-shaded hbase-server hbase-mapreduce 
hbase-backup . hbase-assembly hbase-it U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27221) Bump spotless version to 2.24.1

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579935#comment-17579935
 ] 

Hudson commented on HBASE-27221:


Results for branch branch-2.5
[build #186 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/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}


> Bump spotless version to 2.24.1
> ---
>
> Key: HBASE-27221
> URL: https://issues.apache.org/jira/browse/HBASE-27221
> Project: HBase
>  Issue Type: Task
>  Components: build, pom
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> To catch up with the lastest version.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27299) Bump minimum hadoop 2 version to 2.10.2

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27299?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579936#comment-17579936
 ] 

Hudson commented on HBASE-27299:


Results for branch branch-2.5
[build #186 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/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}


> Bump minimum hadoop 2 version to 2.10.2
> ---
>
> Key: HBASE-27299
> URL: https://issues.apache.org/jira/browse/HBASE-27299
> Project: HBase
>  Issue Type: Task
>  Components: hadoop2, security
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
> Fix For: 2.5.0
>
>
> Although 2.10.2 still have some security issues, at least it is better than 
> other hadoop2 releases and we still want to maintain hadoop 2 support for 
> branch-2.x.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27301) Add Delete addFamilyVersion timestamp verify

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579934#comment-17579934
 ] 

Hudson commented on HBASE-27301:


Results for branch branch-2.5
[build #186 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/]:
 (/) *{color:green}+1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/General_20Nightly_20Build_20Report/]


(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/186/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 Delete addFamilyVersion  timestamp verify
> -
>
> Key: HBASE-27301
> URL: https://issues.apache.org/jira/browse/HBASE-27301
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client
>Reporter: zhengsicheng
>Assignee: zhengsicheng
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27264) Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread Wellington Chevreuil (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wellington Chevreuil updated HBASE-27264:
-
Fix Version/s: 3.0.0-alpha-4

> Add options to consider compressed size when delimiting blocks during hfile 
> writes
> --
>
> Key: HBASE-27264
> URL: https://issues.apache.org/jira/browse/HBASE-27264
> Project: HBase
>  Issue Type: New Feature
>Affects Versions: 3.0.0-alpha-4
>Reporter: Wellington Chevreuil
>Assignee: Wellington Chevreuil
>Priority: Major
> Fix For: 3.0.0-alpha-4
>
>
> In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
> property soo that it can allow for the encoded size to be considered when 
> delimiting hfiles blocks during writes.
> -Here we propose two additional 
> properties,"hbase.block.size.limit.compressed" and  
> "hbase.block.size.max.compressed" that would allow for consider the 
> compressed size (if compression is in use) for delimiting blocks during hfile 
> writing. When compression is enabled, certain datasets can have very high 
> compression efficiency, so that the default 64KB block size and 10GB max file 
> size can lead to hfiles with very large number of blocks.- 
> -In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
> switches to compressed size for delimiting blocks, and 
> "hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
> compressed block size, in order to avoid very large uncompressed blocks 
> (defaulting to 320KB).-
> Note: As of 15/08/2022, the original proposal above has been modified to 
> define a pluggable strategy for predicating block compression rate. Please 
> refer to the release notes for more details. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27264) Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread Wellington Chevreuil (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wellington Chevreuil updated HBASE-27264:
-
Affects Version/s: 3.0.0-alpha-4

> Add options to consider compressed size when delimiting blocks during hfile 
> writes
> --
>
> Key: HBASE-27264
> URL: https://issues.apache.org/jira/browse/HBASE-27264
> Project: HBase
>  Issue Type: New Feature
>Affects Versions: 3.0.0-alpha-4
>Reporter: Wellington Chevreuil
>Assignee: Wellington Chevreuil
>Priority: Major
>
> In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
> property soo that it can allow for the encoded size to be considered when 
> delimiting hfiles blocks during writes.
> -Here we propose two additional 
> properties,"hbase.block.size.limit.compressed" and  
> "hbase.block.size.max.compressed" that would allow for consider the 
> compressed size (if compression is in use) for delimiting blocks during hfile 
> writing. When compression is enabled, certain datasets can have very high 
> compression efficiency, so that the default 64KB block size and 10GB max file 
> size can lead to hfiles with very large number of blocks.- 
> -In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
> switches to compressed size for delimiting blocks, and 
> "hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
> compressed block size, in order to avoid very large uncompressed blocks 
> (defaulting to 320KB).-
> Note: As of 15/08/2022, the original proposal above has been modified to 
> define a pluggable strategy for predicating block compression rate. Please 
> refer to the release notes for more details. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27264) Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread Wellington Chevreuil (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wellington Chevreuil updated HBASE-27264:
-
Release Note: 
This modified the block write logic for deciding on block size boundaries. When 
checking for block size limit, block writer now delegates this decision to 
implementations of the BlockCompressedSizePredicator interface. The aim here is 
to allow for pluggable implementations of how block size limit is defined when 
compression is in use. By default, it assumes the 
UncompressedBlockSizePredicator implementation, which doesn't modify the 
current behaviour of closing the block based on its uncompressed size. The 
alternative PreviousBlockCompressionRatePredicator implementation compares the 
uncompressed size against an adjusted block size, calculated using 
uncompressed/compressed sizes from the previous block as: BLOCK_SIZE * 
(uncompressed/compressed).
Implementations of BlockCompressedSizePredicator should be defined by the 
hbase.block.compressed.size.predicator config property.

To avoid very larger block sizes in the case of large compression factor, this 
also defines the hbase.block.max.size.uncompressed config property. If not set, 
this max size defaults to BLOCK_SIZE * 10.
  Resolution: Fixed
  Status: Resolved  (was: Patch Available)

> Add options to consider compressed size when delimiting blocks during hfile 
> writes
> --
>
> Key: HBASE-27264
> URL: https://issues.apache.org/jira/browse/HBASE-27264
> Project: HBase
>  Issue Type: New Feature
>Reporter: Wellington Chevreuil
>Assignee: Wellington Chevreuil
>Priority: Major
>
> In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
> property soo that it can allow for the encoded size to be considered when 
> delimiting hfiles blocks during writes.
> -Here we propose two additional 
> properties,"hbase.block.size.limit.compressed" and  
> "hbase.block.size.max.compressed" that would allow for consider the 
> compressed size (if compression is in use) for delimiting blocks during hfile 
> writing. When compression is enabled, certain datasets can have very high 
> compression efficiency, so that the default 64KB block size and 10GB max file 
> size can lead to hfiles with very large number of blocks.- 
> -In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
> switches to compressed size for delimiting blocks, and 
> "hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
> compressed block size, in order to avoid very large uncompressed blocks 
> (defaulting to 320KB).-
> Note: As of 15/08/2022, the original proposal above has been modified to 
> define a pluggable strategy for predicating block compression rate. Please 
> refer to the release notes for more details. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27264) Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread Wellington Chevreuil (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wellington Chevreuil updated HBASE-27264:
-
Description: 
In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
property soo that it can allow for the encoded size to be considered when 
delimiting hfiles blocks during writes.

-Here we propose two additional properties,"hbase.block.size.limit.compressed" 
and  "hbase.block.size.max.compressed" that would allow for consider the 
compressed size (if compression is in use) for delimiting blocks during hfile 
writing. When compression is enabled, certain datasets can have very high 
compression efficiency, so that the default 64KB block size and 10GB max file 
size can lead to hfiles with very large number of blocks.- 

-In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
switches to compressed size for delimiting blocks, and 
"hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
compressed block size, in order to avoid very large uncompressed blocks 
(defaulting to 320KB).-

Note: As of 15/08/2022, the original proposal above has been modified to define 
a pluggable strategy for predicating block compression rate. Please refer to 
the release notes for more details. 

 

  was:
In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
property soo that it can allow for the encoded size to be considered when 
delimiting hfiles blocks during writes.

Here we propose two additional properties,"hbase.block.size.limit.compressed" 
and  "hbase.block.size.max.compressed" that would allow for consider the 
compressed size (if compression is in use) for delimiting blocks during hfile 
writing. When compression is enabled, certain datasets can have very high 
compression efficiency, so that the default 64KB block size and 10GB max file 
size can lead to hfiles with very large number of blocks. 

In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
switches to compressed size for delimiting blocks, and 
"hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
compressed block size, in order to avoid very large uncompressed blocks 
(defaulting to 320KB).

 


> Add options to consider compressed size when delimiting blocks during hfile 
> writes
> --
>
> Key: HBASE-27264
> URL: https://issues.apache.org/jira/browse/HBASE-27264
> Project: HBase
>  Issue Type: New Feature
>Reporter: Wellington Chevreuil
>Assignee: Wellington Chevreuil
>Priority: Major
>
> In HBASE-27232 we had modified "hbase.writer.unified.encoded.blocksize.ratio" 
> property soo that it can allow for the encoded size to be considered when 
> delimiting hfiles blocks during writes.
> -Here we propose two additional 
> properties,"hbase.block.size.limit.compressed" and  
> "hbase.block.size.max.compressed" that would allow for consider the 
> compressed size (if compression is in use) for delimiting blocks during hfile 
> writing. When compression is enabled, certain datasets can have very high 
> compression efficiency, so that the default 64KB block size and 10GB max file 
> size can lead to hfiles with very large number of blocks.- 
> -In this proposal, "hbase.block.size.limit.compressed" is a boolean flag that 
> switches to compressed size for delimiting blocks, and 
> "hbase.block.size.max.compressed" is an int with the limit, in bytes for the 
> compressed block size, in order to avoid very large uncompressed blocks 
> (defaulting to 320KB).-
> Note: As of 15/08/2022, the original proposal above has been modified to 
> define a pluggable strategy for predicating block compression rate. Please 
> refer to the release notes for more details. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] wchevreuil merged pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


wchevreuil merged PR #4675:
URL: https://github.com/apache/hbase/pull/4675


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4698: HBASE-27238 backport backup restore to 2.x

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4698:
URL: https://github.com/apache/hbase/pull/4698#issuecomment-1215835380

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 53s |  Docker mode activated.  |
   | -0 :warning: |  yetus  |   0m  5s |  Unprocessed flag(s): 
--brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list 
--whitespace-tabs-ignore-list --quick-hadoopcheck  |
   ||| _ Prechecks _ |
   ||| _ branch-2.5 Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 20s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 46s |  branch-2.5 passed  |
   | +1 :green_heart: |  compile  |   1m 39s |  branch-2.5 passed  |
   | +1 :green_heart: |  shadedjars  |   4m  0s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 49s |  branch-2.5 passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 23s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 42s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 42s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 57s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 55s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  | 232m  5s |  root in the patch failed.  |
   |  |   | 258m 32s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4698 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 3ec3a9c3c445 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | branch-2.5 / 7f6f184f06 |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   | unit | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/testReport/
 |
   | Max. process+thread count | 5025 (vs. ulimit of 3) |
   | modules | C: hbase-protocol-shaded hbase-server hbase-mapreduce 
hbase-backup . hbase-assembly hbase-it U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27154) Backport missing MOB related changes to branch-2

2022-08-15 Thread Huaxiang Sun (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27154?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579895#comment-17579895
 ] 

Huaxiang Sun commented on HBASE-27154:
--

#4617 is merged and backported to branch-2.5 as well.

> Backport missing MOB related changes to branch-2
> 
>
> Key: HBASE-27154
> URL: https://issues.apache.org/jira/browse/HBASE-27154
> Project: HBase
>  Issue Type: Bug
>  Components: mob
>Affects Versions: 2.6.0
>Reporter: Szabolcs Bukros
>Assignee: Andrew Kyle Purtell
>Priority: Major
> Fix For: 2.5.0
>
>
> While trying to backport https://issues.apache.org/jira/browse/HBASE-26969 to 
> branch-2 I have found that multiple major MOB related changes are missing. 
> This change is required for FileBased SFT correctness so the changes it 
> depends on should be backported first. Also any improvement to MOB stability 
> is usually welcomed.
> The missing changes I have found so far:
> https://issues.apache.org/jira/browse/HBASE-22749
> https://issues.apache.org/jira/browse/HBASE-23723
> https://issues.apache.org/jira/browse/HBASE-24163
> There is also a docs change describing the new MOB functionality. But 
> considering that the book is always generated based on master I think it is 
> safe to skip backporting it.
> https://issues.apache.org/jira/browse/HBASE-23198
> I'm planning to backport these changes one by one until we reach a state 
> where HBASE-26969  can be backported too.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] huaxiangsun merged pull request #4704: HBASE-23723 Ensure MOB compaction works in optimized mode after snaps…

2022-08-15 Thread GitBox


huaxiangsun merged PR #4704:
URL: https://github.com/apache/hbase/pull/4704


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27109) Move replication queue storage from zookeeper to a separated HBase table

2022-08-15 Thread Hudson (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579872#comment-17579872
 ] 

Hudson commented on HBASE-27109:


Results for branch HBASE-27109/table_based_rqs
[build #2 on 
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/HBASE-27109%252Ftable_based_rqs/2/]:
 (x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/HBASE-27109%252Ftable_based_rqs/2/General_20Nightly_20Build_20Report/]




(x) {color:red}-1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/HBASE-27109%252Ftable_based_rqs/2/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]


(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11 
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/HBASE-27109%252Ftable_based_rqs/2/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}


> Move replication queue storage from zookeeper to a separated HBase table
> 
>
> Key: HBASE-27109
> URL: https://issues.apache.org/jira/browse/HBASE-27109
> Project: HBase
>  Issue Type: New Feature
>  Components: Replication
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>
> This is a more specific issue based on the works which are already done in 
> HBASE-15867.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4691: HBASE-20904 Prometheus /metrics http endpoint for monitoring

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4691:
URL: https://github.com/apache/hbase/pull/4691#issuecomment-1215617799

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 56s |  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 44s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 45s |  master passed  |
   | +1 :green_heart: |  compile  |   1m 52s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m 49s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 23s |  master passed  |
   | -0 :warning: |  patch  |   7m 38s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 10s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 37s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 48s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 48s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   4m 39s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 20s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  | 374m 58s |  root in the patch failed.  |
   |  |   | 402m 10s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4691 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux d4e78ea239e2 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | unit | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt
 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/testReport/
 |
   | Max. process+thread count | 2355 (vs. ulimit of 3) |
   | modules | C: hbase-common hbase-hadoop-compat hbase-http . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4707: HBASE-27303 Unnecessary replication to secondary region replicas shou…

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4707:
URL: https://github.com/apache/hbase/pull/4707#issuecomment-1215568739

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m  4s |  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  |   2m 31s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 39s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   3m 47s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 25s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 19s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 40s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 40s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 47s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 23s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 208m 29s |  hbase-server in the patch passed.  
|
   |  |   | 225m 54s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4707 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux cc27b0161cde 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 
20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/testReport/
 |
   | Max. process+thread count | 2540 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-26982) Add index and bloom filter statistics of LruBlockCache on rs web UI

2022-08-15 Thread Michael Stack (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-26982?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579846#comment-17579846
 ] 

Michael Stack commented on HBASE-26982:
---

Below is image that shows what this PR adds:

 

!165520264-e172911b-58c5-4e74-8d61-00bd66524c94.png!

> Add index and bloom filter statistics of LruBlockCache on rs web UI
> ---
>
> Key: HBASE-26982
> URL: https://issues.apache.org/jira/browse/HBASE-26982
> Project: HBase
>  Issue Type: Improvement
>  Components: BlockCache, UI
>Reporter: Xuesen Liang
>Assignee: Xuesen Liang
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4
>
> Attachments: 165520264-e172911b-58c5-4e74-8d61-00bd66524c94.png
>
>
> When _CombinedBlockCache_ is configured, _LruBlockCache_ is used for 
> index/bloom/meta block cache, and _BucketCache_ is used for data block cache. 
> Index and bloom filter statistics on rs web UI will be helpful.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215560076

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 44s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   2m  8s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 35s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m  0s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 21s |  master passed  |
   | -0 :warning: |  patch  |   4m 28s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m  7s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 35s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 35s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 59s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 21s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 202m 38s |  hbase-server in the patch passed.  
|
   |  |   | 218m 42s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux e3c337c64bd5 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/testReport/
 |
   | Max. process+thread count | 2708 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/console
 |
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (HBASE-26982) Add index and bloom filter statistics of LruBlockCache on rs web UI

2022-08-15 Thread Michael Stack (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-26982?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Stack updated HBASE-26982:
--
Attachment: 165520264-e172911b-58c5-4e74-8d61-00bd66524c94.png

> Add index and bloom filter statistics of LruBlockCache on rs web UI
> ---
>
> Key: HBASE-26982
> URL: https://issues.apache.org/jira/browse/HBASE-26982
> Project: HBase
>  Issue Type: Improvement
>  Components: BlockCache, UI
>Reporter: Xuesen Liang
>Assignee: Xuesen Liang
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4
>
> Attachments: 165520264-e172911b-58c5-4e74-8d61-00bd66524c94.png
>
>
> When _CombinedBlockCache_ is configured, _LruBlockCache_ is used for 
> index/bloom/meta block cache, and _BucketCache_ is used for data block cache. 
> Index and bloom filter statistics on rs web UI will be helpful.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-26982) Add index and bloom filter statistics of LruBlockCache on rs web UI

2022-08-15 Thread Michael Stack (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-26982?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Stack updated HBASE-26982:
--
Hadoop Flags: Reviewed
  Resolution: Fixed
  Status: Resolved  (was: Patch Available)

Merged to master and then cherry-picked to branch-2 and branch-2.5. Thanks for 
the nice patch [~liangxs]  and review [~andrew.purt...@gmail.com] 

> Add index and bloom filter statistics of LruBlockCache on rs web UI
> ---
>
> Key: HBASE-26982
> URL: https://issues.apache.org/jira/browse/HBASE-26982
> Project: HBase
>  Issue Type: Improvement
>  Components: BlockCache, UI
>Reporter: Xuesen Liang
>Assignee: Xuesen Liang
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4
>
>
> When _CombinedBlockCache_ is configured, _LruBlockCache_ is used for 
> index/bloom/meta block cache, and _BucketCache_ is used for data block cache. 
> Index and bloom filter statistics on rs web UI will be helpful.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4698: HBASE-27238 backport backup restore to 2.x

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4698:
URL: https://github.com/apache/hbase/pull/4698#issuecomment-1215548583

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 21s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files 
found.  |
   | +0 :ok: |  shelldocs  |   0m  1s |  Shelldocs was not available.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +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.  |
   ||| _ branch-2.5 Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 17s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 28s |  branch-2.5 passed  |
   | -1 :x: |  compile  |   0m 11s |  root in branch-2.5 failed.  |
   | +1 :green_heart: |  checkstyle  |   1m 15s |  branch-2.5 passed  |
   | +1 :green_heart: |  spotless  |   0m 52s |  branch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |  14m  3s |  branch-2.5 passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 36s |  the patch passed  |
   | -1 :x: |  compile  |   0m 10s |  root in the patch failed.  |
   | -0 :warning: |  cc  |   0m 10s |  root in the patch failed.  |
   | -0 :warning: |  javac  |   0m 10s |  root in the patch failed.  |
   | +1 :green_heart: |  checkstyle  |   1m 12s |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  2s |  There were no new shellcheck 
issues.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  xml  |   0m  4s |  The patch has no ill-formed XML 
file.  |
   | +1 :green_heart: |  hadoopcheck  |  18m 47s |  Patch does not cause any 
errors with Hadoop 2.10.1 or 3.1.2 3.2.2 3.3.1.  |
   | +1 :green_heart: |  hbaseprotoc  |   5m 14s |  the patch passed  |
   | +1 :green_heart: |  spotless  |   0m 50s |  patch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |  15m  7s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   1m  1s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  72m 55s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4698 |
   | Optional Tests | dupname asflicense spotless shellcheck shelldocs javac 
hadoopcheck xml compile spotbugs hbaseanti checkstyle cc hbaseprotoc prototool |
   | uname | Linux b0c37259475a 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | branch-2.5 / 7f6f184f06 |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-general-check/output/branch-compile-root.txt
 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | cc | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | javac | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | Max. process+thread count | 133 (vs. ulimit of 3) |
   | modules | C: hbase-protocol-shaded hbase-server hbase-mapreduce 
hbase-backup hbase-it hbase-assembly . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4698/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 shellcheck=0.4.6 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] saintstack merged pull request #4376: HBASE-26982 Add index and bloom filter statistics of LruBlockCache on…

2022-08-15 Thread GitBox


saintstack merged PR #4376:
URL: https://github.com/apache/hbase/pull/4376


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215537676

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 49s |  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  |   2m 45s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 40s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   3m 56s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 24s |  master passed  |
   | -0 :warning: |  patch  |   4m 28s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 42s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 42s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 42s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 54s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 23s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 190m 48s |  hbase-server in the patch passed.  
|
   |  |   | 208m 46s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 65ac2d0d2d03 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/testReport/
 |
   | Max. process+thread count | 2665 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/console
 |
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4707: HBASE-27303 Unnecessary replication to secondary region replicas shou…

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4707:
URL: https://github.com/apache/hbase/pull/4707#issuecomment-1215525651

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 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  |   2m 45s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 41s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   3m 52s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 25s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 25s |  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  |   3m 52s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 22s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 189m 47s |  hbase-server in the patch passed.  
|
   |  |   | 206m 53s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4707 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 6c687e1f9fba 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/testReport/
 |
   | Max. process+thread count | 2663 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] huaxiangsun commented on pull request #4704: HBASE-23723 Ensure MOB compaction works in optimized mode after snaps…

2022-08-15 Thread GitBox


huaxiangsun commented on PR #4704:
URL: https://github.com/apache/hbase/pull/4704#issuecomment-1215453344

   I run the failed test case TestSplitRegionWhileRSCrash locally, it passed. 


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4691: HBASE-20904 Prometheus /metrics http endpoint for monitoring

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4691:
URL: https://github.com/apache/hbase/pull/4691#issuecomment-1215449498

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 22s |  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  9s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   3m  9s |  master passed  |
   | +1 :green_heart: |  compile  |   2m  6s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m  4s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 55s |  master passed  |
   | -0 :warning: |  patch  |   7m 22s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 10s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m  1s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 58s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 58s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 56s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 46s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 280m 30s |  root in the patch passed.  |
   |  |   | 308m 59s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4691 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux c48fdc7664af 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 
20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/testReport/
 |
   | Max. process+thread count | 4357 (vs. ulimit of 3) |
   | modules | C: hbase-common hbase-hadoop-compat hbase-http . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] ndimiduk commented on pull request #4706: Backport "HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format" to branch-2

2022-08-15 Thread GitBox


ndimiduk commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1215386869

   I mean, maybe @BukrosSzabolcs or @busbey can confirm. I'd rather not 
backport changes that are not required.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Resolved] (HBASE-27296) Some Cell's implementation of toString() such as IndividualBytesFieldCell prints out value and tags which is too verbose

2022-08-15 Thread Huaxiang Sun (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27296?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Huaxiang Sun resolved HBASE-27296.
--
Fix Version/s: 2.5.0
   3.0.0-alpha-4
   2.4.14
   Resolution: Fixed

> Some Cell's implementation of toString() such as IndividualBytesFieldCell 
> prints out value and tags which is too verbose
> 
>
> Key: HBASE-27296
> URL: https://issues.apache.org/jira/browse/HBASE-27296
> Project: HBase
>  Issue Type: Improvement
>  Components: logging
>Affects Versions: 2.4.12
>Reporter: Huaxiang Sun
>Assignee: Huaxiang Sun
>Priority: Minor
> Fix For: 2.5.0, 3.0.0-alpha-4, 2.4.14
>
>
> One of users sees cells >10Mb are logged when over limit at their client log. 
> Checked the code, toString() behavior is not consistent, mostly does not 
> include values and tags. Change toString() to exclude tags/value.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] huaxiangsun merged pull request #4703: HBASE-27296 Some Cell's implementation of toString() such as Individu…

2022-08-15 Thread GitBox


huaxiangsun merged PR #4703:
URL: https://github.com/apache/hbase/pull/4703


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] huaxiangsun merged pull request #4702: HBASE-27296 Some Cell's implementation of toString() such as Individu…

2022-08-15 Thread GitBox


huaxiangsun merged PR #4702:
URL: https://github.com/apache/hbase/pull/4702


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] huaxiangsun merged pull request #4701: HBASE-27296 Some Cell's implementation of toString() such as Individu…

2022-08-15 Thread GitBox


huaxiangsun merged PR #4701:
URL: https://github.com/apache/hbase/pull/4701


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (HBASE-27304) Support using IP to expose master/rs servers for some special scenarios

2022-08-15 Thread LiangJun He (Jira)
LiangJun He created HBASE-27304:
---

 Summary: Support using IP to expose master/rs servers for some 
special scenarios
 Key: HBASE-27304
 URL: https://issues.apache.org/jira/browse/HBASE-27304
 Project: HBase
  Issue Type: New Feature
  Components: master, regionserver
Affects Versions: 3.0.0-alpha-4
Reporter: LiangJun He
Assignee: LiangJun He
 Fix For: 3.0.0-alpha-4


In our cloud environment, when accessing the HBase cluster from another cluster 
(such as the Flink cluster), you cannot use the hostname to access the HBase 
cluster, but you can use IP. Because it is possible that the hostname of the 
HBase cluster automatic expansion node is not maintained in the Flink cluster. 
So we should support using IP to expose master/rs servers for this special 
scenarios.

 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (HBASE-27213) Add support for claim queue operation

2022-08-15 Thread Duo Zhang (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579783#comment-17579783
 ] 

Duo Zhang commented on HBASE-27213:
---

When implementing, I found that there are still some corner cases. For example, 
in the design doc, we only make RemovePeerProcedure to wait for SCP, but what 
if we remove a peer and then add a peer with the same peer id immediately? 
Since in the new design, SCP will add the missing queues for a replication 
peer, the newly added peer with the same peer id may mess thing up.

So maybe in the new design, we should try to make the replication peer uniq by 
adding more identifiers other than the user specified peer id, so we could know 
whether it is the same peer.

Will be back soon when I found a solution, maybe we just need to add a 
timestamp or something.

> Add support for claim queue operation
> -
>
> Key: HBASE-27213
> URL: https://issues.apache.org/jira/browse/HBASE-27213
> Project: HBase
>  Issue Type: Sub-task
>  Components: Replication
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Work started] (HBASE-27214) Implement the new replication log cleaner

2022-08-15 Thread LiangJun He (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27214?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Work on HBASE-27214 started by LiangJun He.
---
> Implement the new replication log cleaner
> -
>
> Key: HBASE-27214
> URL: https://issues.apache.org/jira/browse/HBASE-27214
> Project: HBase
>  Issue Type: Sub-task
>Reporter: Duo Zhang
>Assignee: LiangJun He
>Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4376: HBASE-26982 Add index and bloom filter statistics of LruBlockCache on…

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4376:
URL: https://github.com/apache/hbase/pull/4376#issuecomment-1215140758

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 33s |  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 19s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   3m 18s |  master passed  |
   | +1 :green_heart: |  compile  |   1m 19s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   5m 19s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 51s |  master passed  |
   | -0 :warning: |  patch  |   6m 27s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 49s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 14s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 14s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   5m 16s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 43s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   1m 55s |  hbase-common in the patch passed.  
|
   | +1 :green_heart: |  unit  | 212m 47s |  hbase-server in the patch passed.  
|
   |  |   | 238m 27s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4376 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 58ce6d9bf8f8 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 
09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/testReport/
 |
   | Max. process+thread count | 2790 (vs. ulimit of 3) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215135181

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m  1s |  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  |   2m 15s |  master passed  |
   | +1 :green_heart: |  compile  |   2m 13s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   0m 30s |  master passed  |
   | +1 :green_heart: |  spotless  |   0m 38s |  branch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   1m 16s |  master passed  |
   | -0 :warning: |  patch  |   1m 23s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 15s |  the patch passed  |
   | +1 :green_heart: |  compile  |   2m 13s |  the patch passed  |
   | +1 :green_heart: |  javac  |   2m 13s |  the patch passed  |
   | +1 :green_heart: |  checkstyle  |   0m 31s |  the patch passed  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  hadoopcheck  |  11m 28s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.2 3.3.1.  |
   | +1 :green_heart: |  spotless  |   0m 39s |  patch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   1m 25s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m  9s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  31m 32s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | dupname asflicense javac spotbugs hadoopcheck hbaseanti 
spotless checkstyle compile |
   | uname | Linux 4bd6c637d1bd 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | Max. process+thread count | 60 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/10/console
 |
   | versions | git=2.17.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4707: HBASE-27303 Unnecessary replication to secondary region replicas shou…

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4707:
URL: https://github.com/apache/hbase/pull/4707#issuecomment-1215132801

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 19s |  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  |   2m 30s |  master passed  |
   | +1 :green_heart: |  compile  |   2m 23s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   0m 33s |  master passed  |
   | +1 :green_heart: |  spotless  |   0m 41s |  branch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   1m 21s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 13s |  the patch passed  |
   | +1 :green_heart: |  compile  |   2m 14s |  the patch passed  |
   | +1 :green_heart: |  javac  |   2m 14s |  the patch passed  |
   | +1 :green_heart: |  checkstyle  |   0m 31s |  the patch passed  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  hadoopcheck  |  11m 35s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.2 3.3.1.  |
   | +1 :green_heart: |  spotless  |   0m 38s |  patch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   1m 23s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m 12s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  32m 51s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4707 |
   | Optional Tests | dupname asflicense javac spotbugs hadoopcheck hbaseanti 
spotless checkstyle compile |
   | uname | Linux 91c6415151d0 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | Max. process+thread count | 67 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4707/1/console 
|
   | versions | git=2.17.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4376: HBASE-26982 Add index and bloom filter statistics of LruBlockCache on…

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4376:
URL: https://github.com/apache/hbase/pull/4376#issuecomment-1215127554

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 43s |  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 13s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 55s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 59s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m 15s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 40s |  master passed  |
   | -0 :warning: |  patch  |   5m  8s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 37s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 59s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 59s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   4m 11s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 37s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   1m 53s |  hbase-common in the patch passed.  
|
   | +1 :green_heart: |  unit  | 202m 53s |  hbase-server in the patch passed.  
|
   |  |   | 225m 31s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4376 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 4f8c547c10d4 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 
11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/testReport/
 |
   | Max. process+thread count | 2741 (vs. ulimit of 3) |
   | modules | C: hbase-common hbase-server U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4376/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (HBASE-27279) Make SslHandler work with SaslWrapHandler/SaslUnwrapHandler

2022-08-15 Thread Andor Molnar (Jira)


[ 
https://issues.apache.org/jira/browse/HBASE-27279?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17579755#comment-17579755
 ] 

Andor Molnar commented on HBASE-27279:
--

I think it still makes sense. I'm going to dig and continue my JWT 
implementation and given that it's a new SASL authentication method, it should 
work together with SslHandler. No point using it with Kerberos, but other SASL 
wrappers could benefit from it.

Thanks [~zhangduo] !

> Make SslHandler work with SaslWrapHandler/SaslUnwrapHandler
> ---
>
> Key: HBASE-27279
> URL: https://issues.apache.org/jira/browse/HBASE-27279
> Project: HBase
>  Issue Type: Improvement
>  Components: IPC/RPC, security
>Reporter: Duo Zhang
>Assignee: Duo Zhang
>Priority: Major
>
> Though this is not recommended as SslHandler has already done wrap/unwrap, it 
> is still better to make it work(maybe we could add some warning logs to say 
> this is unnecessary?) than fail with strange error message.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215087889

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 59s |  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  |   2m 14s |  master passed  |
   | +1 :green_heart: |  compile  |   2m 12s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   0m 30s |  master passed  |
   | +1 :green_heart: |  spotless  |   0m 39s |  branch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   1m 15s |  master passed  |
   | -0 :warning: |  patch  |   1m 21s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | -1 :x: |  mvninstall  |   1m 14s |  root in the patch failed.  |
   | -1 :x: |  compile  |   1m 15s |  hbase-server in the patch failed.  |
   | -0 :warning: |  javac  |   1m 15s |  hbase-server in the patch failed.  |
   | +1 :green_heart: |  checkstyle  |   0m 29s |  the patch passed  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | -1 :x: |  hadoopcheck  |   1m 21s |  The patch causes 14 errors with 
Hadoop v3.1.2.  |
   | -1 :x: |  hadoopcheck  |   2m 43s |  The patch causes 14 errors with 
Hadoop v3.2.2.  |
   | -1 :x: |  hadoopcheck  |   4m  6s |  The patch causes 14 errors with 
Hadoop v3.3.1.  |
   | +1 :green_heart: |  spotless  |   0m 36s |  patch has no errors when 
running spotless:check.  |
   | -1 :x: |  spotbugs  |   0m 27s |  hbase-server in the patch failed.  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m  8s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  17m  8s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | dupname asflicense javac spotbugs hadoopcheck hbaseanti 
spotless checkstyle compile |
   | uname | Linux 9ae751a1263a 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | mvninstall | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-mvninstall-root.txt
 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-compile-hbase-server.txt
 |
   | javac | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-compile-hbase-server.txt
 |
   | hadoopcheck | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-javac-3.1.2.txt
 |
   | hadoopcheck | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-javac-3.2.2.txt
 |
   | hadoopcheck | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-javac-3.3.1.txt
 |
   | spotbugs | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-general-check/output/patch-spotbugs-hbase-server.txt
 |
   | Max. process+thread count | 64 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/console 
|
   | versions | git=2.17.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215087354

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 12s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   2m 50s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 47s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   3m 51s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 27s |  master passed  |
   | -0 :warning: |  patch  |   4m 26s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | -1 :x: |  mvninstall  |   1m 35s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m 46s |  hbase-server in the patch failed.  |
   | -0 :warning: |  javac  |   0m 46s |  hbase-server in the patch failed.  |
   | -1 :x: |  shadedjars  |   3m  7s |  patch has 14 errors when building our 
shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 27s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   0m 49s |  hbase-server in the patch failed.  |
   |  |   |  16m 45s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux ad054304246c 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   | mvninstall | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt
 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt
 |
   | javac | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt
 |
   | shadedjars | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt
 |
   | unit | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/testReport/
 |
   | Max. process+thread count | 78 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4675: HBASE-27264 Add options to consider compressed size when delimiting blocks during hfile writes

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4675:
URL: https://github.com/apache/hbase/pull/4675#issuecomment-1215084805

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 50s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   2m 23s |  master passed  |
   | +1 :green_heart: |  compile  |   0m 34s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m  2s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 24s |  master passed  |
   | -0 :warning: |  patch  |   4m 32s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | -1 :x: |  mvninstall  |   1m  9s |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m 34s |  hbase-server in the patch failed.  |
   | -0 :warning: |  javac  |   0m 34s |  hbase-server in the patch failed.  |
   | -1 :x: |  shadedjars  |   3m  6s |  patch has 14 errors when building our 
shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 22s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  |   0m 35s |  hbase-server in the patch failed.  |
   |  |   |  14m 42s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4675 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 1621206a977c 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | mvninstall | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/patch-mvninstall-root.txt
 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt
 |
   | javac | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt
 |
   | shadedjars | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/patch-shadedjars.txt
 |
   | unit | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/testReport/
 |
   | Max. process+thread count | 64 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4675/9/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Assigned] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei reassigned HBASE-27303:


Assignee: chenglei

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Assignee: chenglei
>Priority: Major
>
> As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} throws an exception, we 
> should abort the region server to avoid the data inconsistent between the 
> primary region and secondary region replicas, so we have no need to replicate 
> to secondary region replicas when  {{WAL.sync}} throws an exception.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Status: Patch Available  (was: Open)

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Assignee: chenglei
>Priority: Major
>
> As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} throws an exception, we 
> should abort the region server to avoid the data inconsistent between the 
> primary region and secondary region replicas, so we have no need to replicate 
> to secondary region replicas when  {{WAL.sync}} throws an exception.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] comnetwork opened a new pull request, #4707: HBASE-27303 Unnecessary replication to secondary region replicas shou…

2022-08-15 Thread GitBox


comnetwork opened a new pull request, #4707:
URL: https://github.com/apache/hbase/pull/4707

   …ld avoid when WAL.sync throws Exception


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Description: As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} throws an 
exception, we should abort the region server to avoid the data inconsistent 
between the primary region and secondary region replicas, so we have no need to 
replicate to secondary region replicas when  {{WAL.sync}} throws an exception.  
(was: As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} get an exception, we 
should abort the region server to avoid the data inconsistent between the 
primary region and secondary region replicas, so we have no need to replicate 
to )

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} throws an exception, we 
> should abort the region server to avoid the data inconsistent between the 
> primary region and secondary region replicas, so we have no need to replicate 
> to secondary region replicas when  {{WAL.sync}} throws an exception.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Description: As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} get an 
exception, we should abort the region server to avoid the data inconsistent 
between the primary region and secondary region replicas, so we have no need to 
replicate to   (was: As HBASE-27223 said, )

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27230 and HBASE-27223 said, if {{WAL.sync}} get an exception, we 
> should abort the region server to avoid the data inconsistent between the 
> primary region and secondary region replicas, so we have no need to replicate 
> to 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Description: As HBASE-27223 said,   (was: As HBASE-27223 said)

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27223 said, 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)
chenglei created HBASE-27303:


 Summary: Unnecessary replication to secondary region replicas 
should avoid when WAL.sync throws Exception
 Key: HBASE-27303
 URL: https://issues.apache.org/jira/browse/HBASE-27303
 Project: HBase
  Issue Type: Improvement
Reporter: chenglei






--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Affects Version/s: 3.0.0-alpha-4

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Affects Versions: 3.0.0-alpha-4
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27223 said



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Component/s: read replicas

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>  Components: read replicas
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27223 said



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (HBASE-27303) Unnecessary replication to secondary region replicas should avoid when WAL.sync throws Exception

2022-08-15 Thread chenglei (Jira)


 [ 
https://issues.apache.org/jira/browse/HBASE-27303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

chenglei updated HBASE-27303:
-
Description: As HBASE-27223 said

> Unnecessary replication to secondary region replicas should avoid when 
> WAL.sync throws Exception
> 
>
> Key: HBASE-27303
> URL: https://issues.apache.org/jira/browse/HBASE-27303
> Project: HBase
>  Issue Type: Improvement
>Reporter: chenglei
>Priority: Major
>
> As HBASE-27223 said



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215049432

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 29s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   3m 17s |  master passed  |
   | +0 :ok: |  refguide  |   3m  6s |  branch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  spotless  |   1m 14s |  branch has no errors when 
running spotless:check.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   3m 12s |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  There were no new shellcheck 
issues.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +0 :ok: |  refguide  |   2m 54s |  patch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  spotless  |   1m  1s |  patch has no errors when 
running spotless:check.  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m 14s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  17m 51s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests | dupname asflicense spotless shellcheck shelldocs refguide 
|
   | uname | Linux 51327528fcf9 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 
11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4305/5/yetus-general-check/output/branch-site/book.html
 |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4305/5/yetus-general-check/output/patch-site/book.html
 |
   | Max. process+thread count | 70 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 shellcheck=0.4.6 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215032321

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 41s |  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 _ |
   ||| _ Patch Compile Tests _ |
   ||| _ Other Tests _ |
   |  |   |   1m 28s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests |  |
   | uname | Linux 24c3d085eac6 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 29 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215031558

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215032385

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 42s |  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 _ |
   ||| _ Patch Compile Tests _ |
   ||| _ Other Tests _ |
   |  |   |   1m 31s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests |  |
   | uname | Linux 538fb0037040 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 39 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215031849

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215031501

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/5/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache9 commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215026998

   Ah, maybe I misunderstood your comment. You also mean we open a new issue to 
land the book update? That's fine by me. I could help implementing the PR or 
reviewing the PR, no problem. 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215026406

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 39s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  No case conflicting files 
found.  |
   | +0 :ok: |  shelldocs  |   0m  0s |  Shelldocs was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   ||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   3m 46s |  master passed  |
   | +0 :ok: |  refguide  |   3m 22s |  branch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  spotless  |   1m  6s |  branch has no errors when 
running spotless:check.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   3m 21s |  the patch passed  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  There were no new shellcheck 
issues.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +0 :ok: |  refguide  |   2m 46s |  patch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  spotless  |   0m 50s |  patch has no errors when 
running spotless:check.  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m 13s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  18m 43s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests | dupname asflicense spotless shellcheck shelldocs refguide 
|
   | uname | Linux 695c599fd0f6 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 
11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4305/4/yetus-general-check/output/branch-site/book.html
 |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4305/4/yetus-general-check/output/patch-site/book.html
 |
   | Max. process+thread count | 64 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
|
   | versions | git=2.17.1 maven=3.6.3 shellcheck=0.4.6 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4694: HBASE-27295 Correct the comment of list_deadservers method in admin.rb

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4694:
URL: https://github.com/apache/hbase/pull/4694#issuecomment-1215025366

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m 19s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   2m 59s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 10s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 57s |  the patch passed  |
   | +1 :green_heart: |  javadoc  |   0m  9s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   9m 58s |  hbase-shell in the patch passed.  |
   |  |   |  18m 29s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4694 |
   | Optional Tests | javac javadoc unit |
   | uname | Linux eb4b7ecb78a3 5.4.0-122-generic #138-Ubuntu SMP Wed Jun 22 
15:00:31 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/testReport/
 |
   | Max. process+thread count | 1365 (vs. ulimit of 3) |
   | modules | C: hbase-shell U: hbase-shell |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache9 commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215023996

   > Hmm okay. @Apache9 how about I close 
[HBASE-26907](https://issues.apache.org/jira/browse/HBASE-26907) this PR and 
let you resolve them on 
[HBASE-27294](https://issues.apache.org/jira/browse/HBASE-27294). Do you might 
grabbing the update to the book once that ticket is resolved? Thanks.
   
   For the book update let's open a separated issue? As part of the branch-2.5 
release process.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4706: Backport "HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format" to branch-2

2022-08-15 Thread GitBox


Apache9 commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1215022186

   > > This is for aligning the code?
   > 
   > @Apache9 Yes it's one of the missing patches that @BukrosSzabolcs listed 
on [HBASE-27154](https://issues.apache.org/jira/browse/HBASE-27154).
   
   I mean the problem describe on HBASE-24163 does not exist on branch-2. But 
anyway, no harm to align the code .


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4694: HBASE-27295 Correct the comment of list_deadservers method in admin.rb

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4694:
URL: https://github.com/apache/hbase/pull/4694#issuecomment-1215021985

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 41s |  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 _ |
   | +1 :green_heart: |  mvninstall  |   2m 46s |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m 10s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 33s |  the patch passed  |
   | +1 :green_heart: |  javadoc  |   0m  8s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   7m 32s |  hbase-shell in the patch passed.  |
   |  |   |  14m 52s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4694 |
   | Optional Tests | javac javadoc unit |
   | uname | Linux 62fbd827f2b5 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/testReport/
 |
   | Max. process+thread count | 1509 (vs. ulimit of 3) |
   | modules | C: hbase-shell U: hbase-shell |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4694: HBASE-27295 Correct the comment of list_deadservers method in admin.rb

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4694:
URL: https://github.com/apache/hbase/pull/4694#issuecomment-1215015652

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 41s |  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  |   2m 22s |  master passed  |
   | +1 :green_heart: |  spotless  |   0m 37s |  branch has no errors when 
running spotless:check.  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m  5s |  the patch passed  |
   | +1 :green_heart: |  rubocop  |   0m 13s |  There were no new rubocop 
issues.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  spotless  |   0m 35s |  patch has no errors when 
running spotless:check.  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m  9s |  The patch does not generate 
ASF License warnings.  |
   |  |   |   7m 59s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4694 |
   | Optional Tests | dupname asflicense javac spotless rubocop |
   | uname | Linux 063f3c0a2b94 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | Max. process+thread count | 57 (vs. ulimit of 3) |
   | modules | C: hbase-shell U: hbase-shell |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4694/2/console 
|
   | versions | git=2.17.1 maven=3.6.3 rubocop=0.80.0 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] ndimiduk commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


ndimiduk commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215012744

   Hmm okay. @Apache9 how about I close HBASE-26907 this PR and let you resolve 
them on HBASE-27294. Do you might grabbing the update to the book once that 
ticket is resolved? 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] ndimiduk commented on pull request #4706: Backport "HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format" to branch-2

2022-08-15 Thread GitBox


ndimiduk commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1215014943

   > This is for aligning the code?
   
   @Apache9 Yes it's one of the missing patches that @BukrosSzabolcs listed on 
HBASE-27154.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215010346

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 43s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  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 13s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  spotless  |   0m 36s |  branch has no errors when 
running spotless:check.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m  3s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  shellcheck  |   0m  1s |  There were no new shellcheck 
issues.  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  spotless  |   0m 34s |  patch has no errors when 
running spotless:check.  |
   ||| _ Other Tests _ |
   | +0 :ok: |  asflicense  |   0m  0s |  ASF License check generated no 
output?  |
   |  |   |   2m 54s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4692 |
   | Optional Tests | dupname asflicense spotless shellcheck shelldocs |
   | uname | Linux b3c13335da96 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 33 (vs. ulimit of 3) |
   | modules | C:  U:  |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 shellcheck=0.4.6 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215010197

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 57s |  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 _ |
   ||| _ Patch Compile Tests _ |
   ||| _ Other Tests _ |
   |  |   |   2m  5s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests |  |
   | uname | Linux 312e578bb58a 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 37 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215009780

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 42s |  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 _ |
   ||| _ Patch Compile Tests _ |
   ||| _ Other Tests _ |
   |  |   |   1m 41s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4305 |
   | Optional Tests |  |
   | uname | Linux de954c16e7c7 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 29 (vs. ulimit of 3) |
   | modules | C: . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215009505

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m  1s |  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 15s |  Maven dependency ordering for branch  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m  5s |  Maven dependency ordering for patch  |
   ||| _ Other Tests _ |
   |  |   |   2m  0s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4692 |
   | Optional Tests |  |
   | uname | Linux 9ef168ecda29 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 41 (vs. ulimit of 3) |
   | modules | C:  U:  |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215009083

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215008904

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215008874

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 43s |  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 13s |  Maven dependency ordering for branch  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m  3s |  Maven dependency ordering for patch  |
   ||| _ Other Tests _ |
   |  |   |   1m 35s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4692 |
   | Optional Tests |  |
   | uname | Linux 40ab38a95851 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Max. process+thread count | 32 (vs. ulimit of 3) |
   | modules | C:  U:  |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215008785

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4305/4/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215008122

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215007920

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4692: HBASE-27294 Add new hadoop releases in our hadoop checks

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4692:
URL: https://github.com/apache/hbase/pull/4692#issuecomment-1215007901

   (!) A patch to the testing environment has been detected. 
   Re-executing against the patched versions to perform further tests. 
   The console is at 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4692/3/console 
in case of problems.
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4691: HBASE-20904 Prometheus /metrics http endpoint for monitoring

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4691:
URL: https://github.com/apache/hbase/pull/4691#issuecomment-1215003517

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   1m  6s |  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 11s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   2m 25s |  master passed  |
   | +1 :green_heart: |  compile  |   6m 12s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   1m  2s |  master passed  |
   | +0 :ok: |  refguide  |   2m 30s |  branch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  spotless  |   0m 40s |  branch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   8m 30s |  master passed  |
   | -0 :warning: |  patch  |   0m 44s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 14s |  the patch passed  |
   | -1 :x: |  compile  |   1m 15s |  root in the patch failed.  |
   | -0 :warning: |  javac  |   1m 15s |  root in the patch failed.  |
   | -0 :warning: |  checkstyle  |   1m  2s |  root: The patch generated 4 new 
+ 0 unchanged - 0 fixed = 4 total (was 0)  |
   | +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.  |
   | +0 :ok: |  refguide  |   2m  6s |  patch has no errors when building the 
reference guide. See footer for rendered docs, which you should manually 
inspect.  |
   | +1 :green_heart: |  hadoopcheck  |  11m 48s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.2 3.3.1.  |
   | +1 :green_heart: |  spotless  |   0m 38s |  patch has no errors when 
running spotless:check.  |
   | +1 :green_heart: |  spotbugs  |   9m  6s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m 40s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  57m 32s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4691 |
   | Optional Tests | dupname asflicense javac refguide spotless xml spotbugs 
hadoopcheck hbaseanti checkstyle compile |
   | uname | Linux 6ae399185bfc 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 
20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 9215066e2b |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4691/3/yetus-general-check/output/branch-site/book.html
 |
   | compile | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | javac | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-general-check/output/patch-compile-root.txt
 |
   | checkstyle | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/artifact/yetus-general-check/output/diff-checkstyle-root.txt
 |
   | refguide | 
https://nightlies.apache.org/hbase/HBase-PreCommit-GitHub-PR/PR-4691/3/yetus-general-check/output/patch-site/book.html
 |
   | Max. process+thread count | 138 (vs. ulimit of 3) |
   | modules | C: hbase-common hbase-hadoop-compat hbase-http . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4691/3/console 
|
   | versions | git=2.17.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache9 commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


Apache9 commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1215002626

   > It looks like we still have hadoop-3.1.2 in the quick-check list for 
branch-2.4, I guess we can bump that version. Also, it appears that I have 
added a column regarding branch-2.5 in this patch.
   > 
   > Let me push an updated based the knowledge of 
[HBASE-27148](https://issues.apache.org/jira/browse/HBASE-27148).
   
   Sounds good.
   
   And please see #4692 , where I plan to change the hadoop check versions for 
our active branches, which also implies what hadoop versions we still wantto 
support. I think for 2.4.x, we should not remove the support of a hadoop 
version, but for 2.5+, we can drop the EOL versions and also the versions which 
have critical CVEs.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4706: HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1214998617

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 54s |  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 _ |
   ||| _ branch-2 Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 47s |  branch-2 passed  |
   | +1 :green_heart: |  compile  |   0m 42s |  branch-2 passed  |
   | +1 :green_heart: |  shadedjars  |   4m  6s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 25s |  branch-2 passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 27s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 42s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 42s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   4m  6s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 23s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 184m 27s |  hbase-server in the patch passed.  
|
   |  |   | 203m 23s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4706 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 34b71f7a2224 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 
23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | branch-2 / b83165cf60 |
   | Default Java | AdoptOpenJDK-11.0.10+9 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/testReport/
 |
   | Max. process+thread count | 2759 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] Apache-HBase commented on pull request #4706: HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format

2022-08-15 Thread GitBox


Apache-HBase commented on PR #4706:
URL: https://github.com/apache/hbase/pull/4706#issuecomment-1214997373

   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 54s |  Docker mode activated.  |
   | -0 :warning: |  yetus  |   0m  5s |  Unprocessed flag(s): 
--brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list 
--whitespace-tabs-ignore-list --quick-hadoopcheck  |
   ||| _ Prechecks _ |
   ||| _ branch-2 Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m 22s |  branch-2 passed  |
   | +1 :green_heart: |  compile  |   0m 34s |  branch-2 passed  |
   | +1 :green_heart: |  shadedjars  |   3m 53s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 23s |  branch-2 passed  |
   ||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   2m  4s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 34s |  the patch passed  |
   | +1 :green_heart: |  javac  |   0m 34s |  the patch passed  |
   | +1 :green_heart: |  shadedjars  |   3m 54s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 20s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  | 185m 39s |  hbase-server in the patch passed.  
|
   |  |   | 202m 14s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/4706 |
   | Optional Tests | javac javadoc unit shadedjars compile |
   | uname | Linux 7463f699fa97 5.4.0-1071-aws #76~18.04.1-Ubuntu SMP Mon Mar 
28 17:49:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | branch-2 / b83165cf60 |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   |  Test Results | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/testReport/
 |
   | Max. process+thread count | 2364 (vs. ulimit of 3) |
   | modules | C: hbase-server U: hbase-server |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4706/1/console 
|
   | versions | git=2.17.1 maven=3.6.3 |
   | Powered by | Apache Yetus 0.12.0 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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] ndimiduk commented on pull request #4305: HBASE-26907 Update Hadoop3 versions for JEP 223 compliance

2022-08-15 Thread GitBox


ndimiduk commented on PR #4305:
URL: https://github.com/apache/hbase/pull/4305#issuecomment-1214996836

   It looks like we still have hadoop-3.1.2 in the quick-check list for 
branch-2.4, I guess we can bump that version. Also, it appears that I have 
added a column regarding branch-2.5 in this patch.
   
   Let me push an updated based the knowledge of HBASE-27148.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [hbase] 2005hithlj commented on pull request #4694: HBASE-27295 Correct the comment of list_deadservers method in admin.rb

2022-08-15 Thread GitBox


2005hithlj commented on PR #4694:
URL: https://github.com/apache/hbase/pull/4694#issuecomment-1214994731

   > List dead server or read server? Typo?
   @Apache9  sir.  Is list 'dead' server. I will correct PR/Issue title.


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



  1   2   >