singhpk234 commented on PR #13225: URL: https://github.com/apache/iceberg/pull/13225#issuecomment-5648554210
> But I don't understand what's being proposed Here is what i was proposing : https://github.com/apache/iceberg/pull/18081/changes/c8116f306a97195975c90d7c4bbebc45a6b6e9fd essentially only enable this hybrid mode when client explicitly sets this as catalog prop, we just need an assertion wired through ... Let me know what do you think of the change .. > I can't imagine that just deciding all this now in this PR for the next release is good To recap, folks advocated we need this in prev release 1.11 https://github.com/apache/iceberg/pull/13225#issuecomment-4377161277, but we were not able to make it, it been ~4 months where some of us in the working group, talked through scenarios, came up with a rest spec proposal. I also raised this in community syncs, it was said there can be use cases we should support this in the sdk the hybrid mode. I was searching for use cases. A couple of folks also reached out to me to see if i can help in getting this in 1.12, because presently engines such as Apache Spark can't work at all with RESTCatalog, I have been trying to follow up in this pr Jul 27 .... I understand totally it takes time ! With that being said, I don't think there is anything wrong in being ambitious to get this in a upcoming release, though we should not be making implementation decision in haste .... Thank you for all the work so far on this ! -- 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]
