Skip to content
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

[ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12 #4438

Merged
merged 5 commits into from
Aug 2, 2022

Conversation

gbnam
Copy link
Contributor

@gbnam gbnam commented Jul 30, 2022

What is this PR for?

Update hbase interpreter version from 1.0.0 to 2.4.12

What type of PR is it?

Refactoring

Todos

  • update hbase interpreter version

What is the Jira issue?

How should this be tested?

  • CI should pass

Screenshots (if appropriate)

Questions:

  • Does the licenses files need to update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@gbnam gbnam changed the title Update hbase interpreter version from 1.0.0 to 2.4.12 [ZEPPELIN-5791] Update hbase interpreter version from 1.0.0 to 2.4.12 Jul 30, 2022
@jongyoul
Copy link
Member

Could you please check what happens when you building it?

[INFO] Zeppelin: Angular interpreter ...................... SUCCESS [  0.375 s]
[INFO] Zeppelin: Livy interpreter ......................... SUCCESS [ 13.572 s]
[INFO] Zeppelin: HBase interpreter ........................ FAILURE [ 12.142 s]
[INFO] Zeppelin: JDBC interpreter ......................... SKIPPED
[INFO] Zeppelin: File System Interpreters ................. SKIPPED

@gbnam
Copy link
Contributor Author

gbnam commented Jul 30, 2022

Error occured when maven download some libs blocked http request.
I solved it by editing the maven settings in local env. (https://gist.github.com/vegaasen/1d545aafeda867fcb48ae3f6cd8fd7c7)

Any ideas or related issues to solve this problem?

@gbnam
Copy link
Contributor Author

gbnam commented Aug 1, 2022

I solved error by exclusion glassfish. Please let me know if there is anything else I should do.
Thanks!

@jongyoul
Copy link
Member

jongyoul commented Aug 1, 2022

Thank you for checking it. Let's see how it goes on :-). BTW, hbase-server might not be used by the code level so I guess we can remove the dependency itself.

@gbnam
Copy link
Contributor Author

gbnam commented Aug 2, 2022

Removed hbase-server dependency from hbase interpreter.
Please check the commits. Thanks!

@jongyoul jongyoul merged commit 21fd4c2 into apache:master Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants