[DATA-2210] Remove deprecated mql_query from TabularDataByMQLRequest (v…

…iamrobotics#444)

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

gen/js/app/data/v1/data_pb.d.ts

@@ -401,9 +401,6 @@ export class TabularDataByMQLRequest extends jspb.Message {
   getOrganizationId(): string;
   setOrganizationId(value: string): void;
 
-  getMqlQuery(): string;
-  setMqlQuery(value: string): void;
-
   clearMqlBinaryList(): void;
   getMqlBinaryList(): Array<Uint8Array | string>;
   getMqlBinaryList_asU8(): Array<Uint8Array>;
@@ -424,7 +421,6 @@ export class TabularDataByMQLRequest extends jspb.Message {
 export namespace TabularDataByMQLRequest {
   export type AsObject = {
     organizationId: string,
-    mqlQuery: string,
     mqlBinaryList: Array<Uint8Array | string>,
   }
 }

gen/js/app/data/v1/data_pb.js

@@ -4065,7 +4065,6 @@ proto.viam.app.data.v1.TabularDataByMQLRequest.prototype.toObject = function(opt
 proto.viam.app.data.v1.TabularDataByMQLRequest.toObject = function(includeInstance, msg) {
   var f, obj = {
     organizationId: jspb.Message.getFieldWithDefault(msg, 1, ""),
-    mqlQuery: jspb.Message.getFieldWithDefault(msg, 2, ""),
     mqlBinaryList: msg.getMqlBinaryList_asB64()
   };
 
@@ -4107,10 +4106,6 @@ proto.viam.app.data.v1.TabularDataByMQLRequest.deserializeBinaryFromReader = fun
       var value = /** @type {string} */ (reader.readString());
       msg.setOrganizationId(value);
       break;
-    case 2:
-      var value = /** @type {string} */ (reader.readString());
-      msg.setMqlQuery(value);
-      break;
     case 3:
       var value = /** @type {!Uint8Array} */ (reader.readBytes());
       msg.addMqlBinary(value);
@@ -4151,13 +4146,6 @@ proto.viam.app.data.v1.TabularDataByMQLRequest.serializeBinaryToWriter = functio
       f
     );
   }
-  f = message.getMqlQuery();
-  if (f.length > 0) {
-    writer.writeString(
-      2,
-      f
-    );
-  }
   f = message.getMqlBinaryList_asU8();
   if (f.length > 0) {
     writer.writeRepeatedBytes(
@@ -4186,24 +4174,6 @@ proto.viam.app.data.v1.TabularDataByMQLRequest.prototype.setOrganizationId = fun
 };
 
 
-/**
- * optional string mql_query = 2;
- * @return {string}
- */
-proto.viam.app.data.v1.TabularDataByMQLRequest.prototype.getMqlQuery = function() {
-  return /** @type {string} */ (jspb.Message.getFieldWithDefault(this, 2, ""));
-};
-
-
-/**
- * @param {string} value
- * @return {!proto.viam.app.data.v1.TabularDataByMQLRequest} returns this
- */
-proto.viam.app.data.v1.TabularDataByMQLRequest.prototype.setMqlQuery = function(value) {
-  return jspb.Message.setProto3StringField(this, 2, value);
-};
-
-
 /**
  * repeated bytes mql_binary = 3;
  * @return {!Array<string>}

proto/viam/app/data/v1/data.proto

@@ -203,12 +203,13 @@ message TabularDataBySQLResponse {
 // TabularDataByMQLRequest requests tabular data using an MQL query.
 message TabularDataByMQLRequest {
   string organization_id = 1;
-  // mql_query is deprecated.
-  string mql_query = 2;
   // mql_binary accepts a MongoDB aggregation pipeline as a list of BSON documents, where each
   // document is one stage in the pipeline. The pipeline is run on the "sensorData.readings"
   // namespace, which holds the Viam organization's tabular data.
   repeated bytes mql_binary = 3;
+
+  reserved 2;
+  reserved "mql_query";
 }
 
 // TabularDataByMQLResponse provides unified tabular data and metadata, queried with MQL.