Skip to content

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
<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

@QiangCai QiangCai force-pushed the integration_spark3.5 branch from 739a5b1 to cb96c8b Compare May 18, 2024 13:39
@chenliang613
Copy link
Contributor

please solve some conflicts.

@QiangCai QiangCai force-pushed the integration_spark3.5 branch from e8c4ae8 to 122a544 Compare September 22, 2024 08:49
@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

</properties>
</profile>
<profile>
<id>spark-2.3</id>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is these removal reqired? All spark profile is removed

<httpclient.version>4.5.14</httpclient.version>
<httpcore.version>4.4.16</httpcore.version>
<scala.binary.version>2.12</scala.binary.version>
<scala.version>2.12.18</scala.version>
Copy link
Contributor

Choose a reason for hiding this comment

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

从Spark 3.5开始,好像要依赖Java 21? 你是用Java 21编译的吗

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