uniding commented on code in PR #8598: URL: https://github.com/apache/seatunnel/pull/8598#discussion_r1943890373
########## seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/master/JobMaster.java: ########## @@ -245,7 +245,6 @@ public synchronized void init(long initializationTimestamp, boolean restart) thr try { Thread.currentThread().setContextClassLoader(classLoader); if (!restart - && !logicalDag.isStartWithSavePoint() Review Comment: 1. Execute the following command to start from savepoint, which will result in errors. Please refer to the bug for details seatunnel.sh -c paimon.config -r 938988912087400449 -n m2p 2. After analysis and debugging, it was found that starting from savepoint does not execute the JobMaster::handleSaveMode method, so the handleSchemaSaveMode method of the PaimonSaveModeHandler class is not executed, resulting in a null pointer error when writing paimon 3.The relevant code call stack is shown in the following figure: <img width="572" alt="image" src="https://github.com/user-attachments/assets/57b394d3-6043-43c4-a67f-4c7de8626790" /> <img width="704" alt="image" src="https://github.com/user-attachments/assets/a6b9e5d5-9516-4e4b-8107-d534b32707ec" /> -- 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: commits-unsubscr...@seatunnel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org