Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HBASE-22910 Enable TestMultiVersionConcurrencyControl #531

Merged
merged 1 commit into from
Aug 25, 2019

Conversation

the-sakthi
Copy link
Member

No description provided.

Copy link
Contributor

@saintstack saintstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good one. Why didn't the build run? We should get a test before commit.

@the-sakthi
Copy link
Member Author

I think the pre-commit build was aborted due to HBASE-22913. Next run is in progress, Stack.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 73 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 mvninstall 417 master passed
+1 compile 62 master passed
+1 checkstyle 91 master passed
+1 shadedjars 305 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 40 master passed
0 spotbugs 282 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 280 master passed
_ Patch Compile Tests _
+1 mvninstall 341 the patch passed
+1 compile 67 the patch passed
+1 javac 67 the patch passed
+1 checkstyle 96 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 332 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1058 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 37 the patch passed
+1 findbugs 284 the patch passed
_ Other Tests _
-1 unit 15313 hbase-server in the patch failed.
+1 asflicense 43 The patch does not generate ASF License warnings.
19266
Reason Tests
Failed junit tests hadoop.hbase.namespace.TestNamespaceAuditor
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.master.TestAssignmentManagerMetrics
hadoop.hbase.client.TestAsyncTableGetMultiThreaded
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-531/2/artifact/out/Dockerfile
GITHUB PR #531
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 2328d2016554 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-531@2/out/precommit/personality/provided.sh
git revision master / 94155fe
Default Java 1.8.0_181
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-531/2/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-531/2/testReport/
Max. process+thread count 4909 (vs. ulimit of 10000)
modules C: hbase-server U: hbase-server
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-531/2/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@the-sakthi
Copy link
Member Author

The test passed. Shall we get this in, @saintstack ?

@the-sakthi the-sakthi merged commit a125453 into apache:master Aug 25, 2019
@the-sakthi the-sakthi deleted the hbase-22910 branch August 25, 2019 00:18
@the-sakthi
Copy link
Member Author

Thanks for your review, Stack.

the-sakthi added a commit that referenced this pull request Aug 25, 2019
Signed-off-by: Stack <stack@apache.org>
the-sakthi added a commit that referenced this pull request Aug 25, 2019
Signed-off-by: Stack <stack@apache.org>
the-sakthi added a commit that referenced this pull request Aug 25, 2019
Signed-off-by: Stack <stack@apache.org>
infraio pushed a commit to infraio/hbase that referenced this pull request Aug 17, 2020
symat pushed a commit to symat/hbase that referenced this pull request Feb 17, 2021
Signed-off-by: Stack <stack@apache.org>
(cherry picked from commit 40d30fc)

Change-Id: I41ca30ef119a4bfb6d3cea298103bf05d3b5900d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants