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 2019/07/11 01:46:00 UTC

[GitHub] [incubator-gobblin] arjun4084346 commented on a change in pull request #2663: [GOBBLIN-796] Add support partial updates for flowConfig

arjun4084346 commented on a change in pull request #2663: [GOBBLIN-796] Add support partial updates for flowConfig
URL: https://github.com/apache/incubator-gobblin/pull/2663#discussion_r302335105
 
 

 ##########
 File path: gobblin-restli/gobblin-flow-config-service/gobblin-flow-config-service-server/src/main/java/org/apache/gobblin/service/FlowConfigsV2Resource.java
 ##########
 @@ -119,6 +121,20 @@ public UpdateResponse update(ComplexResourceKey<FlowId, FlowStatusId> key, FlowC
     return this.getFlowConfigResourceHandler().updateFlowConfig(flowId, flowConfig);
   }
 
+  /**
+   * Partial update the flowConfig specified
+   * @param key composite key containing group name and flow name that identifies the flow to update
+   * @param flowConfigPatch patch describing what fields to change
+   * @return {@link UpdateResponse}
+   */
+  @Override
+  public UpdateResponse update(ComplexResourceKey<FlowId, FlowStatusId> key, PatchRequest<FlowConfig> flowConfigPatch) {
+    String flowGroup = key.getKey().getFlowGroup();
 
 Review comment:
   Do we not want partialUpdate in V1 of flowConfig ?

----------------------------------------------------------------
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


With regards,
Apache Git Services