PChou commented on pull request #14891: URL: https://github.com/apache/flink/pull/14891#issuecomment-775585080
@wangyang0918 `ClassPathPackagedProgramRetriever` is an implementation of `PackagedProgramRetrieve` which only define method `getPackagedProgram`, that means callers should only know about `getPackagedProgram`. Caller should only control the behavior of `ClassPathPackagedProgramRetriever` through `ClassPathPackagedProgramRetriever.Builder`. So i prefer doing changes in the `Builder`. When "other mode" access the `ClassPathPackagedProgramRetriever`, `pipelineClassPath` is empty collection by default, that will not affect the mode's behavior ---------------------------------------------------------------- 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]
