austince commented on pull request #15497: URL: https://github.com/apache/flink/pull/15497#issuecomment-815236924
> > To remedy this we would need to reduce the max parallelism after the fact if this case occurs, but that will add further > complexity for a really weird edge-case. > > We could also add the invariant that is currently defined for reactive mode, where the savepoint can set the max parallelism as long it is at least what has been auto-configured, though I agree that this adds yet another bit of complexity to this tiny use case. To see what this might look like, implemented in: f39c534bfa It is now impossible to hit the case where the savepoint rescales the max parallelism to lower than what was used in resource declaration, so this is not exactly what was supported before, but it is a "best-effort" without ripping everything apart to reduce the max parallelism after the fact if the savepoint contains a lower value. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
