[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-09 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on a diff in the pull request: https://github.com/apache/flink/pull/1543#discussion_r52330385 --- Diff: flink-shaded-hadoop/flink-shaded-hadoop2/pom.xml --- @@ -184,10 +184,10 @@ under the License

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-09 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on a diff in the pull request: https://github.com/apache/flink/pull/1543#discussion_r52330359 --- Diff: flink-shaded-hadoop/pom.xml --- @@ -111,6 +111,7 @@ under the License

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-09 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on a diff in the pull request: https://github.com/apache/flink/pull/1543#discussion_r52327591 --- Diff: flink-shaded-hadoop/pom.xml --- @@ -111,6 +111,7 @@ under the License

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-01 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the pull request: https://github.com/apache/flink/pull/1543#issuecomment-178049275 webhdfs api works with this commit but there could be similar classNotFound problems in future. --- If your project is set up for it, you can reply to this

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-01 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the pull request: https://github.com/apache/flink/pull/1543#issuecomment-178048690 seems to be some misunderstanding. That was not the intention. I suggested relocating hadoop classes so that any hadoop classes in the application are loaded

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-02-01 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the pull request: https://github.com/apache/flink/pull/1543#issuecomment-178038559 Hi @StephanEwen how does separating out hadoop classes solve the classloader problem? Since even if the jars are different, they will be loaded in parent

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-01-25 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the pull request: https://github.com/apache/flink/pull/1543#issuecomment-174524448 Sure. Let me try this out locally and I will get back to you. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-01-24 Thread abhishekagarwal87
Github user abhishekagarwal87 commented on the pull request: https://github.com/apache/flink/pull/1543#issuecomment-174306148 There were duplicate exclusions in the pom.xml. This fixes the webhdfs API issue but it doesn't solve the problem entirely. Not every class referenc

[GitHub] flink pull request: FLINK-3271: Include jetty-util in the dist jar

2016-01-24 Thread abhishekagarwal87
GitHub user abhishekagarwal87 opened a pull request: https://github.com/apache/flink/pull/1543 FLINK-3271: Include jetty-util in the dist jar You can merge this pull request into a Git repository by running: $ git pull https://github.com/abhishekagarwal87/flink work