RussellSpitzer commented on code in PR #16131:
URL: https://github.com/apache/iceberg/pull/16131#discussion_r3687491691
##########
core/src/main/java/org/apache/iceberg/rest/responses/LoadTableResponseParser.java:
##########
@@ -70,6 +72,11 @@ public static void toJson(LoadTableResponse response,
JsonGenerator gen) throws
gen.writeEndArray();
}
+ if (!response.readRestrictions().isEmpty()) {
Review Comment:
We've discussed this a lot of times, but basically the answer has always
been, the administrator of the system must only allow clients which would apply
or fail when this is present.
A client which doesn't know about this field will always ignore it, there is
nothing we can do about that. Any flag or other thing we put in the V1 response
would have the same effect (ignored by older V1 clients who would also ignore
the read restrictions.) A client which does know read restrictions should fail
if it can't read the restrictions. V2 can tie this to the response itself but
for V1 this is pretty much it.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
