Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/4920
  
    My first thought is that moving libraries around is maybe not the best 
approach.
    Also, this seems like it implements a mix between explicitly configuring 
the Hadoop Classpath and implicitly picking up env variables.
    
    I would suggest to actually take a step back, make a proper design doc how 
we want this to work. If we add configuration scripts, why not make it an "all 
explicit, no magic" approach?
    
    Minor comments:
      - `bin/hadoop-classpath` seems like the wrong destination. Such files 
should be in `conf`.


---

Reply via email to