[GitHub] [incubator-streampark] wolfboys commented on pull request #2725: [Bugfix]Upgrade commons cli version to resolve compatibility problem under flink 1.17

["wolfboys (via GitHub)" <gi...@apache.org>]

wolfboys commented on PR #2725:
URL: https://github.com/apache/incubator-streampark/pull/2725#issuecomment-1541350495

   > Hi @cygnusdark , Thanks for your contribution. We tested what's you report and cannot reproduce this issue. Actually, StreamPark uses proxy classloader to submit flink job, whick means the classes are loaded from `$FLINK_HOME/lib`, It's highly possible that some dependency in your `$FLINK_HOME/lib` conflict with flink-dist.
   > 
   > Though we cannot reproduce this problem, this pr that bumps commons-cli version to 1.5.0 looks good to me.
   
   We can expand the scope of testing, for example from Flink 1.12 to 1.17, to further verify compatibility. If the tests pass, I would be happy to merge this pull request.
   
   


-- 
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-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org