KurtYoung commented on a change in pull request #11711:
[FLINK-17103][table-planner-blink] Supports dynamic options for Blink…
URL: https://github.com/apache/flink/pull/11711#discussion_r407355643
##########
File path:
flink-table/flink-table-common/src/main/java/org/apache/flink/table/catalog/CatalogTable.java
##########
@@ -46,4 +46,11 @@
* @return a map of properties
*/
Map<String, String> toProperties();
+
+ /**
+ * Returns a copy of this {@code CatalogTable} with given table options
{@code options}.
+ *
+ * @return a new copy of this table with replaced table options
+ */
+ CatalogTable copy(Map<String, String> options);
Review comment:
use `properties` to be aligned with other places?
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
