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:
[email protected]


Reply via email to