JinkunLiu commented on code in PR #26196:
URL: https://github.com/apache/flink/pull/26196#discussion_r1972678736
##########
docs/content/docs/dev/table/sourcesSinks.md:
##########
@@ -335,8 +335,8 @@ the `ParallelismProvider` interface.
<td>Enables to write partitioned data in a
<code>DynamicTableSink</code>.</td>
</tr>
<tr>
- <td>{{< gh_link
file="flink-table/flink-table-common/src/main/java/org/apache/flink/table/connector/source/abilities/SupportsBucketing.java"
name="SupportsBucketing" >}}</td>
- <td>Enables bucketing for a <code>DynamicTableSink</code>.</td>
+ <td>{{< gh_link
file="flink-table/flink-table-common/src/main/java/org/apache/flink/table/connector/sink/abilities/SupportsBucketing.java"
name="SupportsBucketing" >}}</td>
+ <td>Enables to write bucketed data into a
<code>DynamicTableSink</code>.</td>
Review Comment:
Thanks for your suggestion。
The reason for my modification is that in the context, the descriptions of
other Sink Abilities are consistently phrased as "Enables to do something into
a <code>DynamicTableSink</code>." To maintain consistency, I made this change.
Hope you can take this into consideration and see if there might be a better
way to phrase this here?
Of course, this is just a advice too, and I’ll defer to your decision.
--
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
