mbalassi commented on pull request #18144: URL: https://github.com/apache/flink/pull/18144#issuecomment-1000322021
Hi @viirya, Thanks for the detailed answer and the flexibility in your approach. You clarified a valid point: if both the jobmanager and the taskmanager image already have the Hadoop config baked in we could technically just use that instead of enforcing a configmap being available with this information. This seemingly contradicts my previous position (that environment specific configuration should not be in the container image), but if we think that it streamlines initial user experience or valid use cases as you mention and is reasonably easy to implement it might make sense to do so. I will play with this a bit during the holiday season - let us catch up afterwards, and more importantly happy holidays. 😊 -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
