MiLk commented on pull request #14629:
URL: https://github.com/apache/flink/pull/14629#issuecomment-772181752


   Thank you for your work
   
   Following the discussion on the user ML at the beginning of the week, I 
tried your PR against my workload.
   My goal was to set the ephemeral storage requests and limits, but it seems 
the output of the `ResourceRequirementsBuilder` takes precedence over the 
template, so my config is ignored.
   However I was able to set a `priorityClassName` without trouble.
   
   As discussed in this PR, I think that while options set using flink options 
should have higher precedence, they should be merged whenever possible with the 
template.


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


Reply via email to