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 ed06bc6b539eea115375640703eb0934328daca6
Author: Meng Zhu <[email protected]>
AuthorDate: Tue May 21 16:07:41 2019 +0200

    Added `repeated QuotaConfig` to `QuotaStatus`.
    
    Also marked the `infos` field as deprecated.
    
    `QuotaStatus` is returned by `GET_QUOTA` and `GET /quota`.
    As we introduce quota limits, a new mesage `QuotaConfig`
    is introduced to describe the quota configuration. For
    backwards compatibility, we will fill in both fields
    until `QuotaInfo` is removed (in Mesos 2.0).
    
    Review: https://reviews.apache.org/r/70690
---
 include/mesos/quota/quota.proto    |  3 ++-
 include/mesos/v1/quota/quota.proto | 13 +++++++++++--
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/include/mesos/quota/quota.proto b/include/mesos/quota/quota.proto
index 682c0cc..36a6327 100644
--- a/include/mesos/quota/quota.proto
+++ b/include/mesos/quota/quota.proto
@@ -59,5 +59,6 @@ message QuotaConfig {
  * See v1 `QuotaStatus`.
  */
 message QuotaStatus {
-  repeated QuotaInfo infos = 1;
+  repeated QuotaInfo infos = 1 [deprecated = true];
+  repeated QuotaConfig configs = 2;
 }
diff --git a/include/mesos/v1/quota/quota.proto 
b/include/mesos/v1/quota/quota.proto
index b109ca2..38abd4c 100644
--- a/include/mesos/v1/quota/quota.proto
+++ b/include/mesos/v1/quota/quota.proto
@@ -133,7 +133,16 @@ message QuotaConfig {
  * /quota/status response and `GET_QUOTA` `master::Response`.
  */
 message QuotaStatus {
-  // Returns all non-default quotas. Those ommitted from this
+  // Returns all non-default quotas. Those omitted from this
   // list have the default of: no guarantee and no limit.
-  repeated QuotaInfo infos = 1;
+  //
+  // For backwards compatibility, `infos` are still filled along
+  // side `configs`. Note, however `infos` will contain less information
+  // comparing to `configs` (e.g. info does not contain information
+  // regarding quota limits).
+  repeated QuotaInfo infos = 1 [deprecated = true];
+
+  // Returns all non-default quotas. Those omitted from this
+  // list have the default of: no guarantee and no limit.
+  repeated QuotaConfig configs = 2;
 }

Reply via email to