TD-304: Transform body type into turnover specific metric (#19)

* Deprecate weakly defined limit body type.
* Stop accepting body type in "legacy" configurator

Co-authored-by: Артем <WWW_cool@inbox.ru>

Author: keynslug

proto/configurator.thrift

@@ -9,7 +9,6 @@ typedef base.ID LimitName
 typedef limiter_config.LimitConfigID LimitConfigID
 typedef limiter_config.ShardSize ShardSize
 typedef limiter_config.LimitConfig LimitConfig
-typedef limiter_config.LimitBodyType LimitBodyType
 typedef limiter_config.OperationLimitBehaviour OperationLimitBehaviour
 
 struct LimitCreateParams {
@@ -18,7 +17,6 @@ struct LimitCreateParams {
     /** Идентификатор набора настроек создаваемого лимата, в будущем идентификатор заменит структура конфигурации */
     3: optional LimitName name
     4: optional string description
-    5: optional LimitBodyType body_type
     6: optional OperationLimitBehaviour op_behaviour
 }
 

proto/limiter_config.thrift

@@ -17,7 +17,6 @@ typedef base.CurrencySymbolicCode CurrencySymbolicCode
 
 struct LimitConfigParams {
     1: required LimitConfigID id
-    2: required LimitBodyType body_type
     3: required Timestamp started_at
     4: required ShardSize shard_size
     5: required time_range.TimeRangeType time_range_type
@@ -32,7 +31,6 @@ struct LimitConfig {
     1: required LimitConfigID id
     2: required string processor_type
     3: required Timestamp created_at
-    4: required LimitBodyType body_type
     5: required Timestamp started_at
     6: required ShardSize shard_size
     7: required time_range.TimeRangeType time_range_type
@@ -41,6 +39,9 @@ struct LimitConfig {
     9: optional LimitScope scope
     10: optional string description
     12: optional OperationLimitBehaviour op_behaviour
+
+    // deprecated
+    4: optional LimitBodyType body_type_deprecated
 }
 
 struct OperationLimitBehaviour {
@@ -55,22 +56,38 @@ union OperationBehaviour {
 struct Subtraction {}
 struct Addition {}
 
-union LimitBodyType {
-    1: LimitBodyTypeAmount amount
-    2: LimitBodyTypeCash cash
+union LimitType {
+    1: LimitTypeTurnover turnover
 }
 
-struct LimitBodyTypeAmount {}
-struct LimitBodyTypeCash {
-    1: required CurrencySymbolicCode currency
+struct LimitTypeTurnover {
+    /**
+     * Metric to account turnover with.
+     * If undefined, equivalent to specifying `LimitTurnoverNumber`.
+     */
+    1: optional LimitTurnoverMetric metric
 }
 
+union LimitTurnoverMetric {
+
+    /**
+     * Measure turnover over number of operations.
+     */
+    1: LimitTurnoverNumber number
+
+    /**
+     * Measure turnover over aggregate amount of operations denominated in a single currency.
+     * In the event operation's currency differs from limit's currency operation will be accounted
+     * with appropriate exchange rate fixed against operation's timestamp.
+     */
+    2: LimitTurnoverAmount amount
 
-union LimitType {
-    1: LimitTypeTurnover turnover
 }
 
-struct LimitTypeTurnover {}
+struct LimitTurnoverNumber {}
+struct LimitTurnoverAmount {
+    1: required CurrencySymbolicCode currency
+}
 
 union LimitScope {
 
@@ -134,3 +151,13 @@ union Change {
 struct CreatedChange {
     1: required LimitConfig limit_config
 }
+
+/// Deprecated definitions
+
+union LimitBodyType {
+    2: LimitBodyTypeCash cash
+}
+
+struct LimitBodyTypeCash {
+    1: required CurrencySymbolicCode currency
+}