tzulitai opened a new pull request #47: [FLINK-16326] [core] Eagerly validate 
strictly required Flink configurations
URL: https://github.com/apache/flink-statefun/pull/47
 
 
   This PR achieves the following:
   - 7850755 .Make it clearer in the base `flink-conf.yaml` template what 
configurations should not be touched. Also add 
`execution.checkpointing.max-concurrent-checkpoints: 1` to the template since 
Stateful Functions currently does not support concurrent checkpoints.
   - 5706462 Add eager validation of the strictly required Flink configs to 
`StatefulFunctionsConfig` class.
   - d8c2739 Adapt the Flink Harness to these changes.
   
   ---
   
   ### Verifying
   
   I've verified this for both Flink Harness executions as well as end-to-end 
containerized executions.
   - For Flink Harness executions, I executed the harness example (which fails 
without d8c2739).
   - For end-to-end executions, I did `mvn clean verify -Prun-e2e-tests`

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


With regards,
Apache Git Services

Reply via email to