Hisoka-X commented on PR #9552: URL: https://github.com/apache/seatunnel/pull/9552#issuecomment-3059997291
> 1. Remove the checkpoint storage creation every time the CheckpointManager is created, and reuse the created checkpoint storage. > https://github.com/apache/seatunnel/blob/3836c97a62079f1e8d7ca7a9f32c02175030bb3a/seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/checkpoint/CheckpointManager.java#L102 > 2. When a job does not have checkpoints enabled, skip reading data from the checkpoint storage, which can reduce unnecessary overhead. > https://github.com/apache/seatunnel/blob/3836c97a62079f1e8d7ca7a9f32c02175030bb3a/seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/checkpoint/CheckpointManager.java#L121 > > and > https://github.com/apache/seatunnel/blob/3836c97a62079f1e8d7ca7a9f32c02175030bb3a/seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/checkpoint/CheckpointManager.java#L245 > > Regarding the logic of skipping checkpoint storage reading, for the sake of subsequent maintainability, we can inject a storage named empty into the checkpoint manager when the checkpoint is disabled, and all read and write operations will have no additional effect. @chestnut-c , Could you help to perform an optimization in this way? -- 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]
