gh-yzou commented on code in PR #1126:
URL: https://github.com/apache/polaris/pull/1126#discussion_r2012958376
##########
regtests/t_pyspark/src/iceberg_spark.py:
##########
@@ -72,7 +72,7 @@ def __enter__(self):
"""Initial method for Iceberg Spark session. Creates a Spark session with
specified configs.
"""
packages = [
- "org.apache.iceberg:iceberg-spark-runtime-3.5_2.12:1.7.1",
+ "org.apache.iceberg:iceberg-spark-runtime-3.5_2.12:1.8.1",
Review Comment:
I think the client backward compatibility should be guaranteed by Iceberg,
so testing with most recent version of Iceberg client I felt should be good
enough for Polaris.
Of course, we can also introduce testing with different Iceberg client
version to be safe, but i agree with @liamzwbao that this should happen in a
different PR, in fact, maybe we can move all changes related
iceberg-spark-runtime to a different PR to address the regression test setup.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
