Skip to content

Conversation

@Apache9
Copy link
Contributor

@Apache9 Apache9 commented May 31, 2021

…d for 3.5.1

@Apache9 Apache9 self-assigned this May 31, 2021
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 8s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 markdownlint 0m 0s markdownlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 10s The patch does not generate ASF License warnings.
1m 28s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Thirdparty-PreCommit/job/PR-54/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #54
Optional Tests dupname asflicense markdownlint
uname Linux 8e1d38c52a15 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux
Build tool maven
git revision master / 9fa54d4
Max. process+thread count 28 (vs. ulimit of 1000)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-Thirdparty-PreCommit/job/PR-54/1/console
versions git=2.20.1
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@busbey
Copy link
Contributor

busbey commented Jun 1, 2021

The 3.5.1 RC is aimed at releasing stuff currently in the master branch? That branch includes HBASE-25855 which I don't see in the release notes; I thought it was a breaking change?

@Apache9
Copy link
Contributor Author

Apache9 commented Jun 2, 2021

The 3.5.1 RC is aimed at releasing stuff currently in the master branch? That branch includes HBASE-25855 which I don't see in the release notes; I thought it was a breaking change?

Let me check.

@Apache9
Copy link
Contributor Author

Apache9 commented Jun 2, 2021

I think it is a bug. And since thirdparty is only used by HBase itself, I think we should include this in 3.5.1 release and it breaks something, we need to fix it.

Let me change the fix version for this issue and regenerate the CHANGES and RELEASENOTES.

Thanks.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 38s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 markdownlint 0m 0s markdownlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 7s The patch does not generate ASF License warnings.
0m 51s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-Thirdparty-PreCommit/job/PR-54/2/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #54
Optional Tests dupname asflicense markdownlint
uname Linux fa98e3f1395b 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 GNU/Linux
Build tool maven
git revision master / 2e9e6ea
Max. process+thread count 28 (vs. ulimit of 1000)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-Thirdparty-PreCommit/job/PR-54/2/console
versions git=2.20.1
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9 Apache9 merged commit f83a47d into apache:master Jun 2, 2021
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.

4 participants