Skip to content

Conversation

@Apache9
Copy link
Contributor

@Apache9 Apache9 commented Feb 25, 2020

No description provided.

@Apache9 Apache9 requested a review from busbey February 25, 2020 13:40
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 32s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HBASE-22514 Compile Tests _
+1 💚 mvninstall 5m 36s HBASE-22514 passed
+0 🆗 refguide 4m 51s branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1 💚 mvninstall 5m 1s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+0 🆗 refguide 4m 45s patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1 💚 asflicense 0m 19s The patch does not generate ASF License warnings.
22m 20s
Subsystem Report/Notes
Docker Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/1/artifact/out/Dockerfile
GITHUB PR #1206
Optional Tests dupname asflicense refguide
uname Linux 61a3f8d1b8e6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1206/out/precommit/personality/provided.sh
git revision HBASE-22514 / 77b24fd
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/1/artifact/out/branch-site/book.html
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/1/artifact/out/patch-site/book.html
Max. process+thread count 96 (vs. ulimit of 10000)
modules C: . U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/1/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache9 Apache9 force-pushed the HBASE-23890 branch 2 times, most recently from 6c08afd to 69d8992 Compare February 26, 2020 12:21
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 36s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HBASE-22514 Compile Tests _
+1 💚 mvninstall 5m 37s HBASE-22514 passed
+0 🆗 refguide 4m 49s branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1 💚 mvninstall 5m 1s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+0 🆗 refguide 4m 43s patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1 💚 asflicense 0m 19s The patch does not generate ASF License warnings.
22m 16s
Subsystem Report/Notes
Docker Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/2/artifact/out/Dockerfile
GITHUB PR #1206
Optional Tests dupname asflicense refguide
uname Linux b9cb15d80f0e 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1206/out/precommit/personality/provided.sh
git revision HBASE-22514 / 1d8e3b7
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/2/artifact/out/branch-site/book.html
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/2/artifact/out/patch-site/book.html
Max. process+thread count 76 (vs. ulimit of 10000)
modules C: . U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/2/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 34s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HBASE-22514 Compile Tests _
+1 💚 mvninstall 6m 52s HBASE-22514 passed
+0 🆗 refguide 7m 58s branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1 💚 mvninstall 7m 46s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+0 🆗 refguide 8m 22s patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1 💚 asflicense 0m 29s The patch does not generate ASF License warnings.
33m 20s
Subsystem Report/Notes
Docker Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/3/artifact/out/Dockerfile
GITHUB PR #1206
Optional Tests dupname asflicense refguide
uname Linux 1b3281eb1438 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1206/out/precommit/personality/provided.sh
git revision HBASE-22514 / 1d8e3b7
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/3/artifact/out/branch-site/book.html
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/3/artifact/out/patch-site/book.html
Max. process+thread count 96 (vs. ulimit of 10000)
modules C: . U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/3/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache9 Apache9 self-assigned this Feb 27, 2020
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 52s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ HBASE-22514 Compile Tests _
+1 💚 mvninstall 6m 15s HBASE-22514 passed
+0 🆗 refguide 5m 58s branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1 💚 mvninstall 5m 52s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+0 🆗 refguide 5m 32s patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1 💚 asflicense 0m 17s The patch does not generate ASF License warnings.
26m 59s
Subsystem Report/Notes
Docker Client=19.03.6 Server=19.03.6 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/4/artifact/out/Dockerfile
GITHUB PR #1206
Optional Tests dupname asflicense refguide
uname Linux 45c44fa894b5 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1206/out/precommit/personality/provided.sh
git revision HBASE-22514 / 1d8e3b7
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/4/artifact/out/branch-site/book.html
refguide https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/4/artifact/out/patch-site/book.html
Max. process+thread count 86 (vs. ulimit of 10000)
modules C: . U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1206/4/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache9
Copy link
Contributor Author

Apache9 commented Feb 28, 2020

Any other concerns? @busbey

Copy link
Contributor

@busbey busbey left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for putting this together.

@Apache9 Apache9 merged commit 4a94451 into apache:HBASE-22514 Feb 29, 2020
asfgit pushed a commit that referenced this pull request Feb 29, 2020
Signed-off-by: Sean Busbey <busbey@apache.org>
Apache9 added a commit that referenced this pull request Mar 2, 2020
Signed-off-by: Sean Busbey <busbey@apache.org>
thangTang pushed a commit to thangTang/hbase that referenced this pull request Apr 16, 2020
Signed-off-by: Sean Busbey <busbey@apache.org>
thangTang pushed a commit to thangTang/hbase that referenced this pull request Apr 16, 2020
Signed-off-by: Sean Busbey <busbey@apache.org>
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