kl0u commented on pull request #12791:
URL: https://github.com/apache/flink/pull/12791#issuecomment-660953616


   Hi @eogramow , my only comments have to do with refactorings and you can 
find them 
[here](https://github.com/kl0u/flink/tree/FLINK-18362-updated-rebased), or I 
can do the changes while merging. 
   
   For the test, why not having something like the tests in the 
`testPerJobWithProvidedLibDirs`, where the job has a "zip" file shipped to the 
cluster and the job reads the contents in the `open()` of a rich function. 
Could this work?


----------------------------------------------------------------
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]


Reply via email to