github-actions[bot] commented on PR #10359:
URL: https://github.com/apache/iceberg/pull/10359#issuecomment-2467006389
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If
github-actions[bot] closed pull request #10359: Spec: Add context query
parameter for all REST APIs
URL: https://github.com/apache/iceberg/pull/10359
--
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
github-actions[bot] commented on PR #10359:
URL: https://github.com/apache/iceberg/pull/10359#issuecomment-2453235712
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
jackye1995 commented on PR #10359:
URL: https://github.com/apache/iceberg/pull/10359#issuecomment-2122678361
Thanks for the quick feedback Dan! I think this is a similar discussion to
the MV one for properties vs new metadata, I think at least in this case the
property-based approach fits t
danielcweeks commented on PR #10359:
URL: https://github.com/apache/iceberg/pull/10359#issuecomment-2121093565
I think my main concern with this is that it just feels like a way to
workaround the spec. Even some of the examples are things that should be well
defined (e.g. engine).
I
jackye1995 opened a new pull request, #10359:
URL: https://github.com/apache/iceberg/pull/10359
This PR proposes adding a `context` query parameter for all requests.
This was briefly described as something desirable in
https://docs.google.com/document/d/14nmuxxfzQsYo59o0Fbpb-pxOlzS6bV