This is an automated email from the ASF dual-hosted git repository.

mzhu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 1f35f1ab56d59acf8024746fe52ac5f74535b17d
Author: Meng Zhu <[email protected]>
AuthorDate: Tue May 21 17:06:30 2019 +0200

    Added more comments regarding `message QuotaConfig`.
    
    Review: https://reviews.apache.org/r/70691
---
 include/mesos/v1/quota/quota.proto | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/include/mesos/v1/quota/quota.proto 
b/include/mesos/v1/quota/quota.proto
index 38abd4c..d9d2b3b 100644
--- a/include/mesos/v1/quota/quota.proto
+++ b/include/mesos/v1/quota/quota.proto
@@ -76,7 +76,16 @@ message QuotaRequest {
 }
 
 /**
- * Describes a role's quota configuration.
+ * Describes a role's quota configuration. This replaces the legacy
+ * quota related messages: `QuotaInfo` and `QuotaRequest`. This is
+ * used in both `UPDATE_QUOTA` and `GET_QUOTA` calls as well as persisted
+ * in the registry.
+ *
+ * We use this new message in the new call `UPDATE_QUOTA` instead of reusing
+ * the legacy `QuotaRequest` message. This avoids confusions with deprecated
+ * fields and provides a cleaner interface. In `QuotaStatus` returned by
+ * `GET_QUOTA`, the legacy `infos` field is still filled along side `configs`
+ * for backwards compatibility.
  */
 message QuotaConfig {
   required string role = 1;

Reply via email to