-
Notifications
You must be signed in to change notification settings - Fork 178
HBASE-28790 hbase-connectors fails to build with hbase 2.6.0 #133
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
Conversation
NihalJain
commented
Aug 17, 2024
- Add missing method setRequestAttribute added in HBASE-27657
- Add missing method setRequestAttribute added in HBASE-27657
|
🎊 +1 overall
This message was automatically generated. |
|
Gentle ping @Apache9 |
stoty
left a comment
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.
+1 LGTM
| * This method was added in HBASE-27657. We donot add @Override to allow code to compile with | ||
| * versions of hbase both having and not having HBASE-27657 | ||
| **/ | ||
| public TableBuilder setRequestAttribute(String key, byte[] value) { |
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.
This is a fragile approach. This repository needs sureing up if it is going to intentionally support multiple minor versions of HBase. At the minimum, we need an HBase versions check in our pre-commit and nightlies, similar to what HBase has for Hadoop.
|
Could be modelled on the HBase profiles setup in Phoenix. |
|
@NihalJain could you possibly share the dependency versions you were using to build with HBase 2.6.0? I'm actually trying to get the package to build for another HBase version, but was running into problems with the Hadoop version, 3.2.4 worked (when using HBase 2.5.10), but nothing above that, just wondering what combination ended up working for HBase 2.6.0? |
|
I have built with hadoop 3.3.6 apart from 3.2.4. Which are the build versions you are using? What is the issue? Mind sharing may be I can take a look |
|
@NihalJain thanks for responding so quickly! Here is the branch I am testing: https://github.com/Matulis/hbase-connectors/tree/WIP_Hadoop_3_4_0 I'm trying to pin the versions to match AWS EMR 7.5.0. It looks like the HBase cluster fails to start (I see the same with 3.3.6), not sure if it's possibly related to building on ARM based Mac (I'll try to get someone on an Intel Mac to try the build): |
Thanks for the response. I think the issue is mockito. You have 2 options:
Let me know if this helps. |
|
@NihalJain so I pulled your PR (with no changes it builds), when I try to bump the Hadoop version to 3.3.6 I get the same issue. Do you remember which permutation of dependencies work on 3.3.6 for you? I'm currently using JDK8, but will try with 17 to see if anything changes. Thanks! |