This code is way to confusing. It strikes me as very odd that `args` is never 
null, and the naming scheme is weird (args should be "argsList", "argsStr" 
should be "args" to be somewhat consistent with the method arguments).

I would pass in a single collection as arguments and a simple enum for how 
arguments should be passed.

[ Full content available at: https://github.com/apache/flink/pull/6754 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to