zeodtr commented on PR #54: URL: https://github.com/apache/iceberg-rust/pull/54#issuecomment-1734962223
> > at least made `Option`al. > > Sounds like a good idea to me. > > However, this PR has already been merged. What do you think about creating a new issue to initiate a discussion? I believe we can refine the catalog API in subsequent PRs. It would be nice to create a new issue for this. But the scope of the issue should be decided. Should it be only for the `location` field or more general? Perhaps you can decide the scope and create a new issue for that. Thanks. -- 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]
