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

[TASK][EASY] kyuubi-spark-connector-hive supports Spark 3.5 #5384

Closed
2 of 4 tasks
wForget opened this issue Oct 9, 2023 · 4 comments
Closed
2 of 4 tasks

[TASK][EASY] kyuubi-spark-connector-hive supports Spark 3.5 #5384

wForget opened this issue Oct 9, 2023 · 4 comments

Comments

@wForget
Copy link
Member

wForget commented Oct 9, 2023

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Describe the feature

Currently the kyuubi-spark-connector-hive module is not compatible with spark 3.5.

kyuubi-spark-connector-hive_compatible_with_Spark_3_5.patch

Motivation

No response

Describe the solution

No response

Additional context

No response

Are you willing to submit PR?

  • Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
  • No. I cannot submit a PR at this time.
@pan3793 pan3793 changed the title [FEATURE] Make kyuubi-spark-connector-hive compatible with Spark 3.5 [TASK][EASY] kyuubi-spark-connector-hive supports Spark 3.5 Oct 9, 2023
@ITzhangqiang
Copy link
Contributor

Is there anything else that needs to be done with this issue? I see there is already a patch

@wForget
Copy link
Member Author

wForget commented Oct 11, 2023

Is there anything else that needs to be done with this issue? I see there is already a patch

We may need to make changes compatible with spark 3.5 and older versions. The patch in issue description is only a change for spark 3.5 and may not be compatible with older versions.

@pan3793
Copy link
Member

pan3793 commented Feb 26, 2024

cc @lsm1 @Kwafoor, you guys may be interested in this task 👀

@beryllw
Copy link
Contributor

beryllw commented Mar 4, 2024

interested.
Please assgin to me, thanks.

beryllw added a commit to beryllw/incubator-kyuubi that referenced this issue Mar 6, 2024
beryllw added a commit to beryllw/incubator-kyuubi that referenced this issue Mar 6, 2024
beryllw added a commit to beryllw/incubator-kyuubi that referenced this issue Mar 7, 2024
beryllw added a commit to beryllw/incubator-kyuubi that referenced this issue Mar 7, 2024
pan3793 pushed a commit to beryllw/incubator-kyuubi that referenced this issue Mar 7, 2024
@pan3793 pan3793 closed this as completed in 8cc9b98 Mar 7, 2024
zhaohehuhu pushed a commit to zhaohehuhu/incubator-kyuubi that referenced this issue Mar 21, 2024
# 🔍 Description
## Issue References 🔗

This pull request fixes apache#5384

## Describe Your Solution 🔧

## Types of changes 🔖

- [ ] Bugfix (non-breaking change which fixes an issue)
- [x] 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

---

# Checklist 📝

- [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes apache#6133 from Kwafoor/kyuubi_6073.

Closes apache#5384

9234e35 [Cheng Pan] fix
7766dfd [Cheng Pan] nit
e9da162 [Cheng Pan] nit
676bfb2 [Cheng Pan] pretty
c241859 [Cheng Pan] pretty
0eedcf8 [wangjunbo] compat with spark 3.3
3d86654 [wangjunbo] format code
a0898f5 [wangjunbo] delete Unused import
9577f7f [wangjunbo] [KYUUBI apache#5384] kyuubi-spark-connector-hive supports Spark 3.5

Lead-authored-by: Cheng Pan <chengpan@apache.org>
Co-authored-by: wangjunbo <wangjunbo@qiyi.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants