danielcweeks commented on code in PR #12050:
URL: https://github.com/apache/iceberg/pull/12050#discussion_r1926140195
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2945,6 +2946,14 @@ components:
items:
type: integer
+ EnableRowLineageUpdate:
Review Comment:
Comparing to the other updates, this is the first case where we're enabling
a feature via update. Does it make sense to call this `RowLineageUpdate`
without the `Enable` prefix so that it could be use to enable/disable? I feel
like we're setting a precedent here and it's not clear that want to add
multiple updates just to differentiate between enable/disable.
--
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...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
