Skip to content

Commit f882f67

Browse files
authored
fix(AIP-134): consolidate AIP-161 mask guidance (#1173)
1 parent 95a9860 commit f882f67

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

aip/general/0134.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ message UpdateBookRequest {
9090
- The field **must** identify the [resource type][aip-123] of the resource
9191
being updated.
9292
- If partial resource update is supported, a field mask **must** be included.
93-
It **must** be of type `google.protobuf.FieldMask`, and it **should** be
93+
It **must** be of type `google.protobuf.FieldMask`, and it **must** be
9494
called `update_mask`.
9595
- The fields used in the field mask correspond to the resource being updated
9696
(not the request message).

aip/general/0161.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ without waiting for UI or client updates.
2121

2222
## Guidance
2323

24-
Update methods (AIP-134) **must** include an `update_mask` field. These
25-
are called "field masks", and they use the `google.protobuf.FieldMask`
26-
type.
24+
These masks of field names are called "field masks". Fields representing a field
25+
mask **must** use the `google.protobuf.FieldMask` type. Field masks are most
26+
common on Update requests (AIP-134).
2727

2828
Field masks **must** always be relative to the resource:
2929

0 commit comments

Comments
 (0)