-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add Hbase new version 2.4.17 and compression #494
Conversation
# Conflicts: # CHANGELOG.md
- Upload HBase sources to Nexus as well - Patch HBase build to fix HBASE-27860 - Custom build HBase against Hadoop 3.3.6
- Include some patches for 2.4.12 but it doesn't build yet due to another issue (HBASE-25983 probably) - Remove unneeded packages Note: The builder part is a bit torn to pieces due to debugging and needs to be cleaned up
When we have more than one image that requires patching it probably makes sense to move |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The images build successfully. I left some comments and will have a look at the related operator PR again
I get the following error when running the kuttl smoke test with the operator from stackabletech/hbase-operator#403
|
Sorry should have mentioned this. Images are not build yet, please try with:
|
I built the images myself. |
I believe the problem is that the environment variable |
The env var is not the problem (but should probably be set anyway). The classpath only contains the following:
and is missing the important part (
|
The |
And the error is that the HEAD version doesn't include the
|
After adding Phoenix to the image, the smoke test fails because apparently Phoenix needs
|
Thank you! That one should be easy to fix, when you do, please add a comment what it is needed for so we don't forget! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The integration tests now work. The comments regarding the handling of the JAVA version will be added to new issue
Merge for now, track improvements here #495 |
Description
fixes #332
Definition of Done Checklist
Tasks