alpreu commented on pull request #17819: URL: https://github.com/apache/flink/pull/17819#issuecomment-998722383
> What's the benefit of having an enum? Initially, I thought of just implementing factory methods in `AbstractKubernetesParameters` which generate a String based on the internally stored cluster ID. I'm wondering whether there is some reason where we would benefit from adding an enum here. But then once again you have to touch the maximum length every time a parameter is added in the future, that was what I wanted to avoid -- 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]
