cuibo01 commented on pull request #18279: URL: https://github.com/apache/flink/pull/18279#issuecomment-1007267850
> Thanks for creating this PR @cuibo01. I think the fix itself is good. I am wondering whether it actually fixes a problem. Did you observe a problem with splitting `-D foo=bar`? If I am not mistaken, then Flink also understands this format. Otherwise the memory configuration would have never worked. thx @tillrohrmann, got, Because jobmanager.memory.jvm-overhead.min is behind the runClass(YarnApplicationClusSocket), it is used as a parameter by runClass, not by System#Property. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
