nielsbasjes commented on issue #11592: URL: https://github.com/apache/flink/pull/11592#issuecomment-616525132
@zentol I have been thinking about everything that was said and I think I found a way that is in line with what was said in the mailing list. In my opinion the file must be present and have valid values. Because these values will be used to load the correct docker image: If it has invalid values it is unsafe to continue as this will lead to bad testing. In line with the remark by @tillrohrmann if the file is missing or badly generated the system will throw a `java.lang.IllegalStateException: The file .flink-runtime.version.properties has not been generated correctly. You MUST run 'mvn generate-sources' in the flink-runtime module.` So the file is either valid or will lead to an exception. Please indicate if this is an acceptable solution. ---------------------------------------------------------------- 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]
