pnowojski commented on pull request #13551: URL: https://github.com/apache/flink/pull/13551#issuecomment-741698338
For one thing, that's a public api, so we wouldn't be able to easily change it. We could maybe mark it partially as `@Internal`, but I don't know how to do it in a clear way. Additionally it could just create confusion for the users, in the worst case they can start using it. Apart from that, if you need to modify production code for testing purposes, it usually means something is wrong with the design (having a testing mini cluster that randomly sets configuration options is a cleaner design). And also relaying on the `System.getProperty` is not the cleanest way. ---------------------------------------------------------------- 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]
