Skip to content

Commit

Permalink
Merge #85251
Browse files Browse the repository at this point in the history
85251: descpb: remove nullable=false option from Reference.IndexIds r=knz,chengxiong-ruan a=michae2

This silences a warning from gogoprotobuf that nullable=false has no
effect for repeated non-nullable native types.

Release note: None

Co-authored-by: Michael Erickson <[email protected]>
  • Loading branch information
craig[bot] and michae2 committed Jul 29, 2022
2 parents 4b7c9bc + e29dc7d commit e10efc8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/sql/catalog/descpb/structured.proto
Original file line number Diff line number Diff line change
Expand Up @@ -1541,8 +1541,8 @@ message FunctionDescriptor {
optional uint32 id = 1 [(gogoproto.nullable) = false,
(gogoproto.customname) = "ID", (gogoproto.casttype) = "ID"];
// If applicable, IDs of the inbound reference table's index.
repeated uint32 index_ids = 2 [(gogoproto.nullable) = false,
(gogoproto.customname) = "IndexIDs", (gogoproto.casttype) = "IndexID"];
repeated uint32 index_ids = 2 [(gogoproto.customname) = "IndexIDs",
(gogoproto.casttype) = "IndexID"];
// If applicable, IDs of the inbound reference table's column.
repeated uint32 column_ids = 3 [(gogoproto.customname) = "ColumnIDs",
(gogoproto.casttype) = "ColumnID"];
Expand Down

0 comments on commit e10efc8

Please sign in to comment.