From 2d881be8c566e4c374061248dd205efd72b7ef4f Mon Sep 17 00:00:00 2001
From: Google APIs <noreply@google.com>
Date: Fri, 12 Apr 2024 13:32:37 -0700
Subject: [PATCH] fix!: mark parent/name fields with the REQUIRED
 field_behavior annotation docs: updated comments

PiperOrigin-RevId: 624277410
---
 .../datatransfer/v1/datatransfer.proto        | 21 ++++++++++++-------
 .../bigquery/datatransfer/v1/transfer.proto   |  8 +++----
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/google/cloud/bigquery/datatransfer/v1/datatransfer.proto b/google/cloud/bigquery/datatransfer/v1/datatransfer.proto
index b5874c991fc8b..16250f8e404d5 100644
--- a/google/cloud/bigquery/datatransfer/v1/datatransfer.proto
+++ b/google/cloud/bigquery/datatransfer/v1/datatransfer.proto
@@ -867,12 +867,15 @@ message StartManualTransferRunsRequest {
     google.protobuf.Timestamp end_time = 2;
   }
 
-  // Transfer configuration name in the form:
+  // Required. Transfer configuration name in the form:
   // `projects/{project_id}/transferConfigs/{config_id}` or
   // `projects/{project_id}/locations/{location_id}/transferConfigs/{config_id}`.
-  string parent = 1 [(google.api.resource_reference) = {
-    type: "bigquerydatatransfer.googleapis.com/TransferConfig"
-  }];
+  string parent = 1 [
+    (google.api.field_behavior) = REQUIRED,
+    (google.api.resource_reference) = {
+      type: "bigquerydatatransfer.googleapis.com/TransferConfig"
+    }
+  ];
 
   // The requested time specification - this can be a time range or a specific
   // run_time.
@@ -900,8 +903,9 @@ message StartManualTransferRunsResponse {
 // A request to enroll a set of data sources so they are visible in the
 // BigQuery UI's `Transfer` tab.
 message EnrollDataSourcesRequest {
-  // The name of the project resource in the form: `projects/{project_id}`
-  string name = 1;
+  // Required. The name of the project resource in the form:
+  // `projects/{project_id}`
+  string name = 1 [(google.api.field_behavior) = REQUIRED];
 
   // Data sources that are enrolled. It is required to provide at least one
   // data source id.
@@ -911,8 +915,9 @@ message EnrollDataSourcesRequest {
 // A request to unenroll a set of data sources so they are no longer visible in
 // the BigQuery UI's `Transfer` tab.
 message UnenrollDataSourcesRequest {
-  // The name of the project resource in the form: `projects/{project_id}`
-  string name = 1;
+  // Required. The name of the project resource in the form:
+  // `projects/{project_id}`
+  string name = 1 [(google.api.field_behavior) = REQUIRED];
 
   // Data sources that are unenrolled. It is required to provide at least one
   // data source id.
diff --git a/google/cloud/bigquery/datatransfer/v1/transfer.proto b/google/cloud/bigquery/datatransfer/v1/transfer.proto
index 13904846ee4f8..bfd775eed2ae2 100644
--- a/google/cloud/bigquery/datatransfer/v1/transfer.proto
+++ b/google/cloud/bigquery/datatransfer/v1/transfer.proto
@@ -117,14 +117,14 @@ message TransferConfig {
     pattern: "projects/{project}/locations/{location}/transferConfigs/{transfer_config}"
   };
 
-  // The resource name of the transfer config.
+  // Identifier. The resource name of the transfer config.
   // Transfer config names have the form either
   // `projects/{project_id}/locations/{region}/transferConfigs/{config_id}` or
   // `projects/{project_id}/transferConfigs/{config_id}`,
   // where `config_id` is usually a UUID, even though it is not
   // guaranteed or required. The name is ignored when creating a transfer
   // config.
-  string name = 1;
+  string name = 1 [(google.api.field_behavior) = IDENTIFIER];
 
   // The desination of the transfer config.
   oneof destination {
@@ -231,11 +231,11 @@ message TransferRun {
     pattern: "projects/{project}/locations/{location}/transferConfigs/{transfer_config}/runs/{run}"
   };
 
-  // The resource name of the transfer run.
+  // Identifier. The resource name of the transfer run.
   // Transfer run names have the form
   // `projects/{project_id}/locations/{location}/transferConfigs/{config_id}/runs/{run_id}`.
   // The name is ignored when creating a transfer run.
-  string name = 1;
+  string name = 1 [(google.api.field_behavior) = IDENTIFIER];
 
   // Minimum time after which a transfer run can be started.
   google.protobuf.Timestamp schedule_time = 3;