Switch to kyuubi-relocated-hive-service-rpc

[pan3793]

🔍 Description

Issue References 🔗

TL;DR there are some issues with shading Thrift RPC classes during the engine packaging phase, see details in the PR description of apache/kyuubi-shaded#20.

Describe Your Solution 🔧

This PR aims to migrate from vanilla hive-service-rpc, libfb303, libthrift to kyuubi-relocated-hive-service-rpc introduced in apache/kyuubi-shaded#20, the detailed works are:

• replace imported deps in pom.xml and rename the package prefix in all modules, except for
  • kyuubi-server there are a few places use vanilla thrift classes to access HMS to get token
  • kyuubi-hive-sql-engine Hive method invocation
• update relocations rules in modules that creates shaded jar
• introduce HiveRpcUtils in kyuubi-hive-sql-engine module for object conversion.

As part of the whole change, this PR upgrades from the Kyuubi Shaded 0.1.0 to 0.2.0, which changes the jars name. see https://kyuubi.apache.org/shaded-release/0.2.0.html

Types of changes 🔖

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests

Pass all Hive UT with Hive 3.1.3, and IT with Hive 3.1.3 and 2.3.9 (also tested with 2.1.1-cdh6.3.2)

---

Checklists

📝 Author Self Checklist

• My code follows the style guidelines of this project
• I have performed a self-review
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

• Pull request title is okay.
• No license issues.
• Milestone correctly set?
• Test coverage is ok
• Assignees are selected.
• Minimum number of approvals
• No changes are requested

Be nice. Be informative.

[codecov-commenter]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (f3f643a) 61.27% compared to head (1d82010) 61.46%.
Report is 2 commits behind head on master.

❗ Current head 1d82010 differs from pull request most recent head b45d4de. Consider uploading reports for the commit b45d4de to get more accurate results

Files	Patch %	Lines
.../org/apache/kyuubi/session/KyuubiSessionImpl.scala	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #5783      +/-   ##
============================================
+ Coverage     61.27%   61.46%   +0.19%     
  Complexity       23       23              
============================================
  Files           608      608              
  Lines         36050    35941     -109     
  Branches       4951     4940      -11     
============================================
+ Hits          22088    22092       +4     
+ Misses        11567    11457     -110     
+ Partials       2395     2392       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pan3793]

also cc @zhouyifan279

[wForget]

LGTM, thanks.

The title description needs to be updated, this pr also switches kyuubi-relocated-zookeeper-34.

[pan3793]

@wForget thanks for checking, I updated the PR description

[wForget]

Do we need to add a comment here to explain why these dependencies are retained?

[pan3793]

sorry I missed this comment, it would be straightforward if the user searches the codebase with the keyword "import org.apache.thrift"

[pan3793]

Thanks, merged to master