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

[WIP][ISSUE-4346] Support Spark 3.5 #4354

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

QiangCai
Copy link
Contributor

Why is this PR needed?

What changes were proposed in this PR?

Does this PR introduce any user interface change?

  • No
  • Yes. (please explain the change and update document)

Is any new testcase added?

  • No
  • Yes

@chenliang613
Copy link
Contributor

Thanks David, start this pull request. please also remove the old spark version integration code.

format/pom.xml Outdated
@@ -37,7 +37,7 @@
<dependency>
<groupId>org.apache.thrift</groupId>
<artifactId>libthrift</artifactId>
<version>0.9.3</version>
<version>0.19.0</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is better we do this in another PR. I am preparing a PR for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@chenliang613
Copy link
Contributor

please solve some conflicts.

@chenliang613
Copy link
Contributor

@QiangCai I saw you submitted 5 commits to community in last week, please let us know the latest progress about this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants