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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to