Skip to content

[GLUTEN-8889][CORE] Bump Spark version from 3.5.2 to 3.5.5#8890

Merged
zhztheplayer merged 9 commits intoapache:mainfrom
jackylee-ch:bump_spark_3.5.5
Aug 14, 2025
Merged

[GLUTEN-8889][CORE] Bump Spark version from 3.5.2 to 3.5.5#8890
zhztheplayer merged 9 commits intoapache:mainfrom
jackylee-ch:bump_spark_3.5.5

Conversation

@jackylee-ch
Copy link
Contributor

What changes were proposed in this pull request?

Support Spark 3.5.5 and upgrade iceberg version to 1.8.1 to fix the compatibility issue with Spark 3.5.4.

How was this patch tested?

GA

@github-actions github-actions bot added CORE works for Gluten Core INFRA TOOLS DOCS labels Mar 4, 2025
@jackylee-ch jackylee-ch marked this pull request as draft March 4, 2025 02:13
@github-actions
Copy link

github-actions bot commented Mar 4, 2025

#8889

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

Run Gluten ClickHouse CI on ARM

@baibaichen
Copy link
Contributor

@jlfsdtc please fix spark 35 pipeline of ch

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

Run Gluten ClickHouse CI on ARM

3 similar comments
@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@baibaichen
Copy link
Contributor

baibaichen commented Mar 13, 2025

@jackylee-ch

Let's first upgrade to 3.5.5 and then upgrade Icerberg? After upgrading to 1.8.1, Icerberg only supports JDK 17, and clickhouse backend need take some time to support.

@jlfsdtc will open pr for upgradeing to 3.5.5.

@jackylee-ch
Copy link
Contributor Author

jackylee-ch commented Mar 13, 2025

Let's first upgrade to 3.5.5 and then upgrade Icerberg? After upgrading to 1.8.1, Icerberg only supports JDK 17, and clickhouse backend need take some time to support.

@baibaichen We need upgrade them both, as Iceberg 1.8.1 fix the core_dump problem with Spark 3.5.4.

@baibaichen
Copy link
Contributor

Let's first upgrade to 3.5.5 and then upgrade Icerberg? After upgrading to 1.8.1, Icerberg only supports JDK 17, and clickhouse backend need take some time to support.

@baibaichen We need upgrade them both, as Iceberg 1.8.1 fix the core_dump problem with Spark 3.5.4.

I understand this,We will try to support JDK 17 in the next month. My suggestion is to upgrade to 3.5.5 first, then use different iceberg versions for different JDKs, and finally, Clickhouse support JDK 17.

Some users are still using JDK 8, and we still need to support them.

@jackylee-ch
Copy link
Contributor Author

I understand this,We will try to support JDK 17 in the next month. My suggestion is to upgrade to 3.5.5 first, then use different iceberg versions for different JDKs, and finally, Clickhouse support JDK 17.

Some users are still using JDK 8, and we still need to support them.

It's a block issue for iceberg with Spark 3.5.5. To upgrade to Spark 3.5.5, we must use Iceberg 1.8.1.

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

<spark.version>3.5.2</spark.version>
<iceberg.version>1.5.0</iceberg.version>
<spark.version>3.5.5</spark.version>
<iceberg.version>1.8.0</iceberg.version>
Copy link
Member

Choose a reason for hiding this comment

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

should we also enable jdk17 here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can’t define JDK 17 here. If we define the java.version to 17 in the profile spark-3.5, then mvn -Pspark-3.5 -Pjava-8 and mvn -Pjava-8 -Pspark-3.5 would exhibit different behaviors.

@zhouyuan
Copy link
Member

I understand this,We will try to support JDK 17 in the next month. My suggestion is to upgrade to 3.5.5 first, then use different iceberg versions for different JDKs, and finally, Clickhouse support JDK 17.
Some users are still using JDK 8, and we still need to support them.

It's a block issue for iceberg with Spark 3.5.5. To upgrade to Spark 3.5.5, we must use Iceberg 1.8.1.

Agreed we better to upgrade on both - unless we drop the iceberg profile on Spark 3.5.5 firstly @baibaichen

@baibaichen
Copy link
Contributor

Agreed we better to upgrade on both - unless we drop the iceberg profile on Spark 3.5.5 firstly @baibaichen

OK

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

1 similar comment
@github-actions
Copy link

github-actions bot commented Apr 8, 2025

Run Gluten ClickHouse CI on ARM

@zhouyuan
Copy link
Member

zhouyuan commented Apr 8, 2025

CK backend added JDk17 support
#9242

@jackylee-ch
Copy link
Contributor Author

@zzcclp I’m still encountering issues with JDK 8.

@zzcclp
Copy link
Contributor

zzcclp commented Aug 13, 2025

@zzcclp I’m still encountering issues with JDK 8.

image There is jdk version issue when using JDK 8

@zzcclp
Copy link
Contributor

zzcclp commented Aug 13, 2025

Run Gluten Clickhouse CI on x86

@zzcclp
Copy link
Contributor

zzcclp commented Aug 13, 2025

The GlutenBroadcastJoinSuite is hanging, the logs are below: a1017d4ce669b2f7e21be2efe275e82e

Please disable this suite first and then run again?

The GlutenSortShuffleSuite also hang, it needs disable first.

@jackylee-ch
Copy link
Contributor Author

image Spark 3.5.5 UT for CH backends has been passed.

@zzcclp
Copy link
Contributor

zzcclp commented Aug 13, 2025

The latest CI will be failed, the GlutenSortShuffleSuite also hang

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@jackylee-ch
Copy link
Contributor Author

The latest CI will be failed, the GlutenSortShuffleSuite also hang

OK, I have disabled this UT to check if it has other problems.

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

1 similar comment
@zzcclp
Copy link
Contributor

zzcclp commented Aug 14, 2025

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

Run Gluten ClickHouse CI on ARM

@jackylee-ch
Copy link
Contributor Author

Run Gluten Clickhouse CI on x86

@jackylee-ch
Copy link
Contributor Author

@zhztheplayer @zzcclp UT has been passed expect some Suites/cases are ignored.

@zhztheplayer
Copy link
Member

Fantastic! @jackylee-ch Thank you for taking care of this millstone constantly in the past few months.

Thank you for all the kindly help with the CH CI! @zzcclp

Without your effort we are going to stay on 3.5.2 permanently :)

@zhztheplayer zhztheplayer merged commit ec9bb99 into apache:main Aug 14, 2025
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants