You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2018/03/14 04:25:03 UTC
[GitHub] yush1ga opened a new pull request #1380: Refactor REST API for namespace policy
yush1ga opened a new pull request #1380: Refactor REST API for namespace policy
URL: https://github.com/apache/incubator-pulsar/pull/1380
### Motivation
#1317
### Modifications
Added common REST API to set/get some namespace policy properties.
### Result
We can get/set some namespace policy properties by common REST API and we don't need to add REST when we add new policy property.
BTW,
In this time, I excluded following properties from common REST API.
- `auth_policies` because it has API for each `auth_policies`' property
- `bundles` because it is split rather than set
- `latency_stats_sample_rate` because it's not used
- `replication_clusters` and `persistence` because these need specific validation method and `persistence`'s get method returns special value when it is null.
If we want to cope with these policy property,
We perhaps should make interface for policy property which have validation, get, set method.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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