[GitHub] [incubator-gobblin] arjun4084346 commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case

arjun4084346 commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-515873361 Just a minor comment. Also one test case is failing, please make sure travis run passes.

[GitHub] [incubator-gobblin] arjun4084346 commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case

arjun4084346 commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-515873047 LGTM. Thanks for the improvements. This i