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

jhsenjaliya commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-537265834 Done, rebased it with GOBBLIN-785, Thanks

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

jhsenjaliya commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-536099696 @sv2000, This PR was created on top of GOBBLIN-785, so can u pls you merge #2650 first ? Thanks --

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

jhsenjaliya commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-536002170 @arjun4084346 , @sv2000 , please take a look at this, this should be ready to get merged now. Thanks -

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

jhsenjaliya commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-521090692 @arjun4084346 , Please review. This is an

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

jhsenjaliya commented on issue #2685: [GOBBLIN-822] Upgrade log4j to log4j2 for LogRouting use-case URL: https://github.com/apache/incubator-gobblin/pull/2685#issuecomment-512962792 cant find any specific error in the build, so please review this and i will kick of another build later. Th