You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@gobblin.apache.org by GitBox <gi...@apache.org> on 2020/10/13 19:52:14 UTC

[GitHub] [incubator-gobblin] jack-moseley commented on a change in pull request #3124: [GOBBLIN-1061] Add option to only accept requests to leader node and redirect requests in the client

jack-moseley commented on a change in pull request #3124:
URL: https://github.com/apache/incubator-gobblin/pull/3124#discussion_r504216140



##########
File path: gobblin-runtime/build.gradle
##########
@@ -66,6 +66,7 @@ dependencies {
   compile externalDependency.gson
   compile externalDependency.guava
   compile externalDependency.guice
+  compile externalDependency.helix

Review comment:
       I did try a lot of different locations for these things... the problem with doing that is that `FlowExecutionResource` needs `FlowStatusGenerator` as well, and I can't add gobblin-service as a dependency because gobblin-service depends on gobblin-restli.
   
   Maybe I can try moving `FlowStatusGenerator` to gobblin-restli, that seems like it should be possible.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org