anonymouscodeholic commented on pull request #13756: URL: https://github.com/apache/flink/pull/13756#issuecomment-714998044
> > After this PR it's not clear why does this test need to be an IT. > > The reason is that it requires to set a job jar to build a `PackagedProgram`, and the dummy job jar used in this test case is built in the packing phase. Sorry, I wasn't being clear enough. I meant just by quickly _reading_ the code, it's not obvious. By looking at the associated JIRA ticket and this PR, it is clear. But I think it'd worth mentioning why the test is an IT also in the codebase. ---------------------------------------------------------------- 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: us...@infra.apache.org
