RussellSpitzer commented on code in PR #12050:
URL: https://github.com/apache/iceberg/pull/12050#discussion_r1927116867
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2945,6 +2946,14 @@ components:
items:
type: integer
+ EnableRowLineageUpdate:
Review Comment:
So our discussion on this on the Design doc and original spec changes was
that we didn't ever want to allow someone to disable row-lineage once it was
enabled. Specifically that it was a one-way valve. We did wonder whether or not
it would be worthwhile to make a generic "EnableFeatureUpdate(String Feature)"
to encapsulate any other one way valves we want to add in the future but at the
moment we really couldn't think of any other things ad the Metadata level we
wanted to add that would be like this.
So specifically here, we didn't want to take a parameter because we don't
want to allow disabling at all.
--
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
