You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2021/01/11 01:53:01 UTC

[hbase] branch HBASE-25476 updated (c585789 -> 80ebabd)

This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a change to branch HBASE-25476
in repository https://gitbox.apache.org/repos/asf/hbase.git.


    omit c585789  HBASE-25476 Enable error prone check in pre commit
     add 45ef88d  HBASE-25471 Move RegionScannerImpl out of HRegion (#2853)
     add 7d70141  HBASE-25434 SlowDelete & SlowPut metric value should use updateDelete & updatePut (#2837)
     add a348204  HBASE-25483 set the loadMeta log level to debug (#2859)
     add 5c233e9  HBASE-25459 WAL can't be cleaned in some scenes (#2848)
     add 8f55a2b  HBASE-25487 [create-release] changes.md update broken (#2864)
     add 49aba57  HBASE-25449 'dfs.client.read.shortcircuit' should not be set in hbase-default.xml
     add 84c4033  HBASE-25489 improve performance of --parse-release-tags (#2867)
     add 54eae0f  HBASE-25473 [create-release] checkcompatibility.py failing with "KeyError: 'binary'" (#2862)
     add 4925a64  HBASE-25333 Add maven enforcer rule to ban VisibleForTesting imports (#2854)
     add 686b72c  HBASE-25249 Adding StoreContext (#2800)
     new 80ebabd  HBASE-25476 Enable error prone check in pre commit

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (c585789)
            \
             N -- N -- N   refs/heads/HBASE-25476 (80ebabd)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 dev-support/Jenkinsfile_GitHub                     |   4 +-
 dev-support/checkcompatibility.py                  |  10 +-
 dev-support/create-release/release-util.sh         |  26 +-
 .../git_jira_release_audit.py                      |  20 +-
 dev-support/jenkins_precommit_github_yetus.sh      |   2 +-
 hbase-common/src/main/resources/hbase-default.xml  |   4 +-
 .../hadoop/hbase/TestHBaseConfiguration.java       |  17 +
 .../src/test/resources/hdfs-default.xml            |  20 +-
 .../src/test/resources/hdfs-scr-enabled.xml        |  18 +-
 .../hadoop/hbase/mapreduce/HFileOutputFormat2.java |   5 +-
 .../hbase/master/assignment/RegionStateStore.java  |   3 +-
 .../java/org/apache/hadoop/hbase/mob/MobUtils.java |   6 +-
 .../hadoop/hbase/regionserver/HMobStore.java       |  32 +-
 .../apache/hadoop/hbase/regionserver/HRegion.java  | 754 +-------------------
 .../apache/hadoop/hbase/regionserver/HStore.java   | 258 ++++---
 .../hbase/regionserver/MetricsRegionServer.java    |  12 +-
 .../hadoop/hbase/regionserver/RSRpcServices.java   |   1 -
 .../hbase/regionserver/RegionScannerImpl.java      | 782 +++++++++++++++++++++
 .../regionserver/ReversedRegionScannerImpl.java    |  13 +-
 .../hadoop/hbase/regionserver/StoreContext.java    | 194 +++++
 .../hadoop/hbase/regionserver/StoreUtils.java      |  25 +
 .../regionserver/wal/SequenceIdAccounting.java     |   6 +-
 .../hadoop/hbase/tool/BulkLoadHFilesTool.java      |   6 +-
 .../wal/BoundedRecoveredHFilesOutputSink.java      |   6 +-
 .../org/apache/hadoop/hbase/io/TestHeapSize.java   |   3 +-
 .../TestTransitRegionStateProcedure.java           |   4 +-
 .../hadoop/hbase/regionserver/TestHRegion.java     |   5 +-
 .../regionserver/TestMetricsRegionServer.java      |  10 +-
 .../hbase/regionserver/TestScanWithBloomError.java |   3 +-
 .../regionserver/TestScannerHeartbeatMessages.java |   5 +-
 .../regionserver/TestSecureBulkLoadManager.java    |   4 +-
 .../hbase/regionserver/TestSwitchToStreamRead.java |   6 +-
 .../hadoop/hbase/regionserver/TestWideScanner.java |   2 +-
 .../regionserver/wal/TestSequenceIdAccounting.java |  18 +
 pom.xml                                            |  13 +
 35 files changed, 1320 insertions(+), 977 deletions(-)
 copy hbase-archetypes/hbase-client-project/src/test/resources/hbase-site.xml => hbase-common/src/test/resources/hdfs-default.xml (64%)
 copy hbase-archetypes/hbase-client-project/src/test/resources/hbase-site.xml => hbase-common/src/test/resources/hdfs-scr-enabled.xml (64%)
 create mode 100644 hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionScannerImpl.java
 create mode 100644 hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreContext.java


[hbase] 01/01: HBASE-25476 Enable error prone check in pre commit

Posted by zh...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch HBASE-25476
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit 80ebabd2ab7e265ad6740bc513cca719c598ddcd
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Mon Jan 11 09:52:01 2021 +0800

    HBASE-25476 Enable error prone check in pre commit
---
 dev-support/Jenkinsfile_GitHub                | 4 +++-
 dev-support/jenkins_precommit_github_yetus.sh | 5 ++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/dev-support/Jenkinsfile_GitHub b/dev-support/Jenkinsfile_GitHub
index d253867..a725f1d 100644
--- a/dev-support/Jenkinsfile_GitHub
+++ b/dev-support/Jenkinsfile_GitHub
@@ -38,7 +38,7 @@ pipeline {
         YETUS_DRIVER_REL = "${SRC_REL}/dev-support/jenkins_precommit_github_yetus.sh"
         // Branch or tag name.  Yetus release tags are 'rel/X.Y.Z'
         YETUS_VERSION = 'rel/0.12.0'
-        GENERAL_CHECK_PLUGINS = 'all,-compile,-javac,-javadoc,-jira,-shadedjars,-unit'
+        GENERAL_CHECK_PLUGINS = 'all,-javadoc,-jira,-shadedjars,-unit'
         JDK_SPECIFIC_PLUGINS = 'compile,github,htmlout,javac,javadoc,maven,mvninstall,shadedjars,unit'
         // output from surefire; sadly the archive function in yetus only works on file names.
         ARCHIVE_PATTERN_LIST = 'TEST-*.xml,org.apache.h*.txt,*.dumpstream,*.dump'
@@ -168,6 +168,7 @@ pipeline {
                         BUILD_URL_ARTIFACTS = "artifact/${WORKDIR_REL}/${PATCH_REL}"
                         DOCKERFILE = "${WORKDIR}/${DOCKERFILE_REL}"
                         YETUS_DRIVER = "${WORKDIR}/${YETUS_DRIVER_REL}"
+                        SKIP_ERRORPRONE = true
                     }
                     steps {
                         dir("${SOURCEDIR}") {
@@ -268,6 +269,7 @@ pipeline {
                         BUILD_URL_ARTIFACTS = "artifact/${WORKDIR_REL}/${PATCH_REL}"
                         DOCKERFILE = "${WORKDIR}/${DOCKERFILE_REL}"
                         YETUS_DRIVER = "${WORKDIR}/${YETUS_DRIVER_REL}"
+                        SKIP_ERRORPRONE = true
                     }
                     steps {
                         dir("${SOURCEDIR}") {
diff --git a/dev-support/jenkins_precommit_github_yetus.sh b/dev-support/jenkins_precommit_github_yetus.sh
index 1c489d6..5bb2b1b 100755
--- a/dev-support/jenkins_precommit_github_yetus.sh
+++ b/dev-support/jenkins_precommit_github_yetus.sh
@@ -122,7 +122,10 @@ YETUS_ARGS+=("--whitespace-tabs-ignore-list=.*/generated/.*")
 YETUS_ARGS+=("--tests-filter=${TESTS_FILTER}")
 YETUS_ARGS+=("--personality=${SOURCEDIR}/dev-support/hbase-personality.sh")
 YETUS_ARGS+=("--quick-hadoopcheck")
-YETUS_ARGS+=("--skip-errorprone")
+if [[ "${SKIP_ERRORPRONE}" = "true" ]]; then
+  # skip error prone
+  YETUS_ARGS+=("--skip-errorprone")
+fi
 # effectively treat dev-support as a custom maven module
 YETUS_ARGS+=("--skip-dirs=dev-support")
 # For testing with specific hadoop version. Activates corresponding profile in maven runs.