[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-10 Thread mjsax
GitHub user mjsax opened a pull request: https://github.com/apache/flink/pull/903 [FLINK-2337] Multiple SLF4J bindings using Storm compatibility layer exclude logback from Storm dependencies You can merge this pull request into a Git repository by running: $ git pull https://gi

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-10 Thread mjsax
Github user mjsax commented on the pull request: https://github.com/apache/flink/pull/903#issuecomment-120479371 Travis failed due to unrelated error: Failed tests: ProcessFailureStreamingRecoveryITCase>AbstractProcessFailureRecoveryTest.testTaskManagerProcessFa

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-11 Thread hsaputra
Github user hsaputra commented on the pull request: https://github.com/apache/flink/pull/903#issuecomment-120589347 +1 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature en

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-11 Thread chiwanpark
Github user chiwanpark commented on the pull request: https://github.com/apache/flink/pull/903#issuecomment-120603967 +1 for merging :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-12 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/903#issuecomment-120702067 +1 looks good --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-13 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/903#issuecomment-120924514 Merging this... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] flink pull request: [FLINK-2337] Multiple SLF4J bindings using Sto...

2015-07-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/903 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabl