Skip to content

Commit

Permalink
feat: publish the RequestStats proto (#1400)
Browse files Browse the repository at this point in the history 
* docs: Remove unnecessary comment

PiperOrigin-RevId: 474807836

Source-Link: googleapis/googleapis@dfd68f7

Source-Link: googleapis/googleapis-gen@fe305e0
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZmUzMDVlMDI1MmJjOWE4ODQ2NmUxYjk2YzJmMjQ2NGRkNzI5Yjk3OCJ9

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: publish the RequestStats proto

PiperOrigin-RevId: 476399682

Source-Link: googleapis/googleapis@0c4e682

Source-Link: googleapis/googleapis-gen@ff87f69
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZmY4N2Y2OWFhZDFmMmZmMDY3NzJmOTE3ZWVjYmVhNjQ5Nzk3ZDlkNSJ9

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* add to clirr ignored differences

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Kristen O'Leary <[email protected]>
  • Loading branch information
@gcf-owl-bot @kolea2
3 people committed Sep 28, 2022
1 parent 58406a9 commit e989200
Show file tree
Hide file tree
Showing 33 changed files with 7,361 additions and 635 deletions.
3 changes: 1 addition & 2 deletions ...gtable/src/main/java/com/google/cloud/bigtable/admin/v2/BaseBigtableTableAdminClient.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -922,8 +922,7 @@ public final UnaryCallable<GetTableRequest, Table> getTableCallable() {
* }</pre>
*
* @param table Required. The table to update. The table's `name` field is used to identify the
* table to update. Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]&#42;`
* table to update.
* @param updateMask Required. The list of fields to update. A mask specifying which fields (e.g.
* `deletion_protection`) in the `table` field should be updated. This mask is relative to the
* `table` field, not to the request message. The wildcard (&#42;) path is currently not
Expand Down
24 changes: 0 additions & 24 deletions ...loud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/UpdateTableRequest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,6 @@ public static final com.google.protobuf.Descriptors.Descriptor getDescriptor() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -162,8 +160,6 @@ public boolean hasTable() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -181,8 +177,6 @@ public com.google.bigtable.admin.v2.Table getTable() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand Down Expand Up @@ -633,8 +627,6 @@ public Builder mergeFrom(
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -651,8 +643,6 @@ public boolean hasTable() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -673,8 +663,6 @@ public com.google.bigtable.admin.v2.Table getTable() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -699,8 +687,6 @@ public Builder setTable(com.google.bigtable.admin.v2.Table value) {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -722,8 +708,6 @@ public Builder setTable(com.google.bigtable.admin.v2.Table.Builder builderForVal
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -750,8 +734,6 @@ public Builder mergeTable(com.google.bigtable.admin.v2.Table value) {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -774,8 +756,6 @@ public Builder clearTable() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -792,8 +772,6 @@ public com.google.bigtable.admin.v2.Table.Builder getTableBuilder() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -812,8 +790,6 @@ public com.google.bigtable.admin.v2.TableOrBuilder getTableOrBuilder() {
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand Down
6 changes: 0 additions & 6 deletions ...able-admin-v2/src/main/java/com/google/bigtable/admin/v2/UpdateTableRequestOrBuilder.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,6 @@ public interface UpdateTableRequestOrBuilder
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -45,8 +43,6 @@ public interface UpdateTableRequestOrBuilder
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand All @@ -61,8 +57,6 @@ public interface UpdateTableRequestOrBuilder
* <pre>
* Required. The table to update.
* The table's `name` field is used to identify the table to update.
* Format:
* `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
* </pre>
*
* <code>.google.bigtable.admin.v2.Table table = 1 [(.google.api.field_behavior) = REQUIRED];
Expand Down
6 changes: 2 additions & 4 deletions ...loud-bigtable-admin-v2/src/main/proto/google/bigtable/admin/v2/bigtable_table_admin.proto
View file
Original file line number Diff line number Diff line change
Expand Up @@ -381,8 +381,8 @@ message RestoreTableRequest {
// Name of the backup from which to restore. Values are of the form
// `projects/<project>/instances/<instance>/clusters/<cluster>/backups/<backup>`.
string backup = 3 [(google.api.resource_reference) = {
type: "bigtableadmin.googleapis.com/Backup"
}];
type: "bigtableadmin.googleapis.com/Backup"
}];
}
}

Expand Down Expand Up @@ -597,8 +597,6 @@ message GetTableRequest {
message UpdateTableRequest {
// Required. The table to update.
// The table's `name` field is used to identify the table to update.
// Format:
// `projects/{project}/instances/{instance}/tables/[_a-zA-Z0-9][-_.a-zA-Z0-9]*`
Table table = 1 [(google.api.field_behavior) = REQUIRED];

// Required. The list of fields to update.
Expand Down
12 changes: 11 additions & 1 deletion proto-google-cloud-bigtable-v2/clirr-ignored-differences.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,16 @@
<difference>
<differenceType>7012</differenceType>
<className>com/google/bigtable/v2/*OrBuilder</className>
<method>* has*(*)</method>
<method>* get*(*)</method>
</difference>
<difference>
<differenceType>7012</differenceType>
<className>com/google/bigtable/v2/*OrBuilder</className>
<method>boolean contains*(*)</method>
</difference>
<difference>
<differenceType>7012</differenceType>
<className>com/google/bigtable/v2/*OrBuilder</className>
<method>boolean has*(*)</method>
</difference>
</differences>

0 comments on commit e989200

Please sign in to comment.