dmvk commented on PR #19372: URL: https://github.com/apache/flink/pull/19372#issuecomment-1097751263
> 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. As long as this would be discussed on the mailing list, it should IMO be ok-ish as it's a change to the configuration and doesn't break any APIs. All-in-all if we just go ahead with 2), it should be fine not doing that as users are already use to the behavior (even though it's bit weird in my opinion). > 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. I'm not sure that I correctly interpret this sentence. Are you suggesting that we should use an info level instead of a warning for the message? If yes, I'm OK with that 👍 -- 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]
