walterddr commented on issue #9336: [FLINK-13548][Deployment/YARN]Support priority of the Flink YARN application URL: https://github.com/apache/flink/pull/9336#issuecomment-525485982 > Just a comment for a possible test. I think it would be enough to test that we submit an `ApplicationSubmissionContext` with the priority set. Hence we could have a `YarnClient` test implementation which we pass to the `AbstractYarnClusterDescriptor`. Not sure, though, how much mocked functionality the `YarnClient` would need to be able to trigger the submit application call. Sorry for missing this comment @tillrohrmann . Yes. I think testing that the `ApplicationSubmissionContext` is having the correct priority setting is probably the best way to verify the change.
---------------------------------------------------------------- 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] With regards, Apache Git Services
