XComp commented on pull request #15131: URL: https://github.com/apache/flink/pull/15131#issuecomment-828183451
> @XComp Thanks for your reply. > Docs is updated and latest commit is uploaded. And this commit has made the following changes compared to the last one. > > 1. Update the `security.kerberos.fetch.delegation-token` conf description > 2. Directly use `token.getService` as credential alias, instead of creating a new object, like `Text alias = new Text(token.getService)`. > 3. Rename some variables. > 4. Some minor changes Thanks for the update @zuston . I'm just wondering why you squashed the two commits into one again. Considering that the two changes (service name as alias and new parameter) are independent of each other I would favor @wangyang0918 's proposal of keeping the two commits separate. -- 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]
