Revert "Updating Service and Model Config Protobuf for uint64 Request Priority" (#92)

This reverts commit e304859.

Author: nnshah1

include/triton/common/triton_json.h

@@ -228,6 +228,8 @@ class TritonJson {
       return TRITONJSON_STATUSSUCCESS;
     }
 
+    // FIXME Should have Set* for all types.
+
     // Set/overwrite a signed integer in a value. This changes the
     // type of the value to signed int.
     TRITONJSON_STATUSTYPE SetInt(const int64_t value)
@@ -237,24 +239,6 @@ class TritonJson {
       return TRITONJSON_STATUSSUCCESS;
     }
 
-    // Set/overwrite an unsigned integer in a value. This changes the
-    // type of the value to unsigned int.
-    TRITONJSON_STATUSTYPE SetUInt(const uint64_t value)
-    {
-      rapidjson::Value& v = AsMutableValue();
-      v.SetUint64(value);
-      return TRITONJSON_STATUSSUCCESS;
-    }
-
-    // Set/overwrite a double in a value. This changes the
-    // type of the value to double.
-    TRITONJSON_STATUSTYPE SetDouble(const double value)
-    {
-      rapidjson::Value& v = AsMutableValue();
-      v.SetDouble(value);
-      return TRITONJSON_STATUSSUCCESS;
-    }
-    
     // Set/overwrite a string in a value. This changes the
     // type of the value to string
     TRITONJSON_STATUSTYPE SetString(const std::string& value)

protobuf/grpc_service.proto

@@ -440,12 +440,8 @@ message InferParameter
 {
   //@@  .. cpp:var:: oneof parameter_choice
   //@@
-  //@@     The parameter value can be a string, an int64,
-  //@@     an uint64, a double, or a boolean
-  //@@
-  //@@     Note: double and uint64 are currently
-  //@@           placeholders for future use and
-  //@@           are not supported for custom parameters
+  //@@     The parameter value can be a string, an int64 or
+  //@@     a boolean
   //@@
   oneof parameter_choice
   {
@@ -466,22 +462,6 @@ message InferParameter
     //@@       A string parameter value.
     //@@
     string string_param = 3;
-
-    //@@    .. cpp:var:: double double_param
-    //@@
-    //@@       A double parameter value.
-    //@@
-    //@@       Not supported for custom parameters
-    //@@
-    double double_param = 4;
-
-    //@@    .. cpp:var:: uint64 uint64_param
-    //@@
-    //@@       A uint64 parameter value.
-    //@@
-    //@@       Not supported for custom parameters
-    //@@
-    uint64 uint64_param = 5;
   }
 }
 

protobuf/model_config.proto

@@ -1134,7 +1134,7 @@ message ModelDynamicBatching
   //@@
   bool preserve_ordering = 3;
 
-  //@@  .. cpp:var:: uint64 priority_levels
+  //@@  .. cpp:var:: uint32 priority_levels
   //@@
   //@@     The number of priority levels to be enabled for the model,
   //@@     the priority level starts from 1 and 1 is the highest priority.
@@ -1143,14 +1143,14 @@ message ModelDynamicBatching
   //@@     priority 3, etc. Requests with the same priority level will be
   //@@     handled in the order that they are received.
   //@@
-  uint64 priority_levels = 4;
+  uint32 priority_levels = 4;
 
-  //@@  .. cpp:var:: uint64 default_priority_level
+  //@@  .. cpp:var:: uint32 default_priority_level
   //@@
   //@@     The priority level used for requests that don't specify their
   //@@     priority. The value must be in the range [ 1, 'priority_levels' ].
   //@@
-  uint64 default_priority_level = 5;
+  uint32 default_priority_level = 5;
 
   //@@  .. cpp:var:: ModelQueuePolicy default_queue_policy
   //@@
@@ -1161,13 +1161,13 @@ message ModelDynamicBatching
   //@@
   ModelQueuePolicy default_queue_policy = 6;
 
-  //@@  .. cpp:var:: map<uint64, ModelQueuePolicy> priority_queue_policy
+  //@@  .. cpp:var:: map<uint32, ModelQueuePolicy> priority_queue_policy
   //@@
   //@@     Specify the queue policy for the priority level. The default queue
   //@@     policy will be used if a priority level doesn't specify a queue
   //@@     policy.
   //@@
-  map<uint64, ModelQueuePolicy> priority_queue_policy = 7;
+  map<uint32, ModelQueuePolicy> priority_queue_policy = 7;
 }
 
 //@@