DanielLeens commented on issue #11125:
URL: https://github.com/apache/seatunnel/issues/11125#issuecomment-5453922821

   Thanks for tracing this back to the current branch.
   
   I rechecked current `dev` at `1ff72ceb3d2a823a0fca64ec009984d75e846c28`. The 
original root concern from this issue is already addressed there: 
`FileMapStore` now routes only `engine_runningJobMetrics` through a no-op 
storage path, while the live in-memory metrics flow remains intact and the 
other engine state maps continue to use their normal persistence path.
   
   So for this issue specifically, I would treat `#11244` as covering the 
original disk-pressure problem on `dev`. The fact that the implementation is 
unconditional rather than config-gated does not look like a reason to keep this 
exact issue open.
   
   If the community later wants an opt-in switch to persist running metrics 
again, that should be a separate enhancement discussion, because it reopens 
policy and semantics questions rather than the root issue reported here.
   


-- 
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]

Reply via email to