github-actions[bot] commented on code in PR #69224:
URL: https://github.com/apache/doris/pull/69224#discussion_r4304916747


##########
docker/thirdparties/docker-compose/iceberg/iceberg.yaml.tpl:
##########
@@ -35,8 +35,8 @@ services:
       - ./spark-defaults.conf:/opt/spark/conf/spark-defaults.conf
       - 
./data/input/jars/iceberg-aws-bundle-1.10.1.jar:/opt/spark/jars/iceberg-aws-bundle-1.10.1.jar
       - 
./data/input/jars/iceberg-spark-runtime-4.0_2.13-1.10.1.jar:/opt/spark/jars/iceberg-spark-runtime-4.0_2.13-1.10.1.jar
-      - 
./data/input/jars/paimon-s3-1.3.1.jar:/opt/spark/jars/paimon-s3-1.3.1.jar
-      - 
./data/input/jars/paimon-spark-4.0-1.3.1.jar:/opt/spark/jars/paimon-spark-4.0-1.3.1.jar
+      - 
./data/input/jars/paimon-s3-1.4.2.jar:/opt/spark/jars/paimon-s3-1.4.2.jar
+      - 
./data/input/jars/paimon-spark-4.0_2.13-1.4.2.jar:/opt/spark/jars/paimon-spark-4.0_2.13-1.4.2.jar

Review Comment:
   [P2] Refresh cached jars before switching these mounts
   
   After an existing Iceberg environment is updated to this revision, running 
`run-thirdparties-docker.sh -c iceberg` retains its `data/` directory and skips 
the new archive download (`start_iceberg`, lines 1440–1457). The previous 
archive contains only the two Paimon 1.3.1 jars, so neither new 1.4.2 source 
path exists. [Compose creates directories for missing short-syntax bind 
sources](https://docs.docker.com/reference/compose-file/services/#volumes), 
leaving Spark without the Paimon catalog classes. Paimon fixture initialization 
and the suite's `spark_paimon` queries then fail to load the configured 
catalog. Please refresh the required jars or detect and invalidate an 
incompatible cached fixture before starting Compose.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to