twalthr commented on pull request #16482: URL: https://github.com/apache/flink/pull/16482#issuecomment-881248782
Thanks for this PR @Airblader. Before I will do a full review, I would kindly ask you to split the first commit into a separate PR and JIRA issue. Configuration is an important part of Flink and more people should look at this change. It looks very good but also touches important classes. In any case, I would mark `DescribedEnum` as at least `@PublicEvolving`. -- 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]
