hello, all, A user has reported a issue[1] about PATCH method of admin API. I looked at the PR[2] that was causing user confusion, and I think the user is using it in the right way and our implementation is inappropriate.
For example, if user want to update the `method` of `/apisix/admin/routes/1`, user need to PATCH with data: `"methods": ["GET", null, null, null, null, null, null, null, null]`. For me, I don't know why I need a lot of `null` after "GET". >From the user's perspective, the current implementation is not appropriate. So I suggest roll back the current PATCH implementation[2] for admin api. what do you think? [1] https://github.com/apache/incubator-apisix/issues/1823 [2] https://github.com/apache/incubator-apisix/pull/1609 [3] https://github.com/apache/incubator-apisix/pull/1609/files#diff-00625723b6e737f3cdb18af67165b70fR996 Thanks, Ming Wen, Apache APISIX(incubating) & Apache SkyWalking Twitter: _WenMing
