[GitHub] [incubator-streampark] lvshaokang commented on issue #2565: [DISCUSSION] refactor v2 api use RESTful API

["lvshaokang (via GitHub)" <gi...@apache.org>]

lvshaokang commented on issue #2565:
URL: https://github.com/apache/incubator-streampark/issues/2565#issuecomment-1499278797

   > @lvshaokang @wolfboys We should change post data with json instead of form. With this issue merged ,I will carry on replace `enum` type issue.
   
   @GOODBOY008 Right. Some controller methods is redundant, but for one or two fields of the method whether to also use json entity, may need to consider it. Some times we can use path variable or query variable to pass parameters, like, 
   [DELETE] /alert-configs/{config-id}
   [GET] /alert-configs?a=b&c=d


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

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

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