architgyl commented on PR #22509: URL: https://github.com/apache/flink/pull/22509#issuecomment-1542652921
> Thanks for the patch. Left some comments. > > Ideally we should have a test case verifying that the ACL actually works. Have you verified the ACL actually works with a real deployment in YARN? @becketqin I have verified locally with the possible scenarios for both the use cases: - view the logs - kill application Also, verified the wildcard behavior. Updated tests related to it and also, added post change behavior in the Testing part of the PR. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
