gaborgsomogyi commented on PR #19372: URL: https://github.com/apache/flink/pull/19372#issuecomment-1097737082
Correct me if I'm wrong but AFAIK Flink must avoid breaking changes between minor releases. Having `true ` as default for `KERBEROS_FETCH_DELEGATION_TOKEN` may or may not be good but is a must if we want to keep compatibility. All in all I'm open to change the default but from user perspective it would require minor migration. Considering this and the fact that there is already a pattern in Flink for bullet point 2 I would vote on that. We can print out a meaningful message what does that actually mean. Just a side not all example are using info level messages so I think it would be good to keep this pattern if you don't have objection. -- 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]
