Skip to content

Conversation

@petersomogyi
Copy link
Contributor

No description provided.

@petersomogyi
Copy link
Contributor Author

This is the version currently used by the hbase-operator-tools repository. The docker build command succeeded locally with this Dockerfile.

# https://builds.apache.org/job/PreCommit-HBASE-CONNECTORS-Build

FROM maven:3.6-jdk-8
FROM maven:3.8-jdk-8
Copy link
Contributor

Choose a reason for hiding this comment

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

so changing this fixes the build right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

# Avoid out of memory errors in builds
###
ENV MAVEN_OPTS -Xmx3g
ENV MAVEN_OPTS=-Xmx3g
Copy link
Contributor

Choose a reason for hiding this comment

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

is this change needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The build showed a warning for legacy formar.

 1 warning found (use docker --debug to expand):
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 34)

@Apache-HBase
Copy link

(!) 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-Connectors-PreCommit/job/PR-147/1/console in case of problems.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 45s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 hadolint 0m 0s hadolint was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 21s Maven dependency ordering for branch
+1 💚 spotless 0m 18s branch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 3s Maven dependency ordering for patch
+1 💚 shellcheck 0m 0s There were no new shellcheck issues.
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 spotless 0m 13s patch has no errors when running spotless:check.
_ Other Tests _
2m 47s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-147/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #147
Optional Tests dupname spotless hadolint shellcheck shelldocs
uname Linux a5660a6ebdd3 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 GNU/Linux
Build tool hb_maven
Personality dev-support/jenkins/hbase-personality.sh
git revision master / 6544291
Max. process+thread count 34 (vs. ulimit of 12500)
modules C: U:
Console output https://ci-hbase.apache.org/job/HBase-Connectors-PreCommit/job/PR-147/1/console
versions git=2.30.2 shellcheck=0.7.1
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@petersomogyi petersomogyi merged commit 08659ba into apache:master Jul 29, 2025
1 check passed
@petersomogyi petersomogyi deleted the HBASE-29476 branch July 29, 2025 05:22
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