zeroshade commented on issue #437: URL: https://github.com/apache/iceberg-go/issues/437#issuecomment-2941744777
Thanks for taking a stab at it. I'll take a look at it tomorrow, just wanna give my initial thoughts here: > We’re faced with two options for addressing the current behavior: > > Decouple the catalog context from later I/O entirely, and shift the responsibility to the PlanFiles context. > > Combine both contexts - one from the catalog and one for PlanFiles - and propagate them together. My preference would be to decouple the catalog context from later I/O entirely. Ideally, the preference would be for the catalog to not store a context 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
