libenchao commented on code in PR #24035:
URL: https://github.com/apache/flink/pull/24035#discussion_r1498571218
##########
docs/content/docs/connectors/table/formats/protobuf.md:
##########
@@ -149,9 +149,12 @@ Format Options
<td style="word-wrap: break-word;">false</td>
<td>Boolean</td>
<td>
- This option only works if the generated class's version is proto2.
If this value is set to true, the format will read empty values as the default
values defined in the proto file.
+ If this value is set to true, the format will read empty values as
the default values defined in the proto file.
If the value is set to false, the format will generate null values
if the data element does not exist in the binary protobuf message.
- If the proto syntax is proto3, this value will forcibly be set to
true, because proto3's standard is to use default values.
+ If the proto syntax is proto3, this value needs to be set to true by
users when using proto version older than 3.15 because proto3
Review Comment:
@sharath1709 Thanks for the reminder, this is indeed a breaking change, we
can add a release note to talk about it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
