wuchong commented on a change in pull request #14792:
URL: https://github.com/apache/flink/pull/14792#discussion_r567671294



##########
File path: docs/dev/table/connectors/formats/canal.zh.md
##########
@@ -189,6 +189,12 @@ metadata fields for its value format.
       <td>The timestamp at which the connector processed the event. 
Corresponds to the <code>ts</code>
       field in the Canal record.</td>
     </tr>
+    <tr>
+      <td><code>event-timestamp</code></td>
+      <td><code>TIMESTAMP(3) WITH LOCAL TIME ZONE NULL</code></td>
+      <td>The timestamp at which the connector produced the event. Corresponds 
to the <code>es</code>

Review comment:
       This description is misleading, because "the connector produced the 
event" sounds "ingetsion time". The event and the `es` is not produced by the 
connector, but the binlog dump of mysql server. Maybe we can say:
   
   ```
   The timestamp when the corresponding change was executed in MySQL server.
   ```




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to