Skip to content

Commit

Permalink
[#11693] Fix partition_key to use bytes
Browse files Browse the repository at this point in the history
Summary:
partition_key was added to PgsqlWriteRequestPB in recent c5f5125/D13244 commit.
The field has type string, but it could contain arbitrary data, so bytes type should be used for it.
This diff fixes the issue by switching to bytes type.

Test Plan: Jenkins

Reviewers: dmitry

Reviewed By: dmitry

Subscribers: bogdan, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D15877
  • Loading branch information
spolitov committed Mar 9, 2022
1 parent 209a80d commit c91783a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/common/pgsql_protocol.proto
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ message PgsqlWriteRequestPB {
repeated PgsqlColRefPB col_refs = 21;

// Used only in pg client.
optional string partition_key = 22;
optional bytes partition_key = 22;
}

//--------------------------------------------------------------------------------------------------
Expand Down

0 comments on commit c91783a

Please sign in to comment.