[GitHub] helix pull request #174: [HELIX-691] Allow users to update InstanceConfig

[zhan849]

Github user zhan849 commented on a diff in the pull request:

    https://github.com/apache/helix/pull/174#discussion_r180178181
  
    --- Diff: 
helix-rest/src/main/java/org/apache/helix/rest/server/resources/helix/InstanceAccessor.java
 ---
    @@ -315,26 +316,27 @@ public Response 
getInstanceConfig(@PathParam("clusterId") String clusterId,
         return notFound();
       }
     
    -  @PUT
    +  @POST
    --- End diff --
    
    PUT is for "set" and POST is for "patch", I'd suggest we keep both. 
@dasahcc thoughts?


---