zeroshade opened a new pull request, #266: URL: https://github.com/apache/iceberg-go/pull/266
Refactoring and restructuring the `catalog` package based on discussions with others in the Go community. This achieves the following benefits: * Improved, and shorter, naming of types and methods. Things that are specific to the REST catalog or Glue catalog can be put directly into those packages allowing for a public API such as `rest.NewCatalog` instead of `catalog.NewRestCatalog`. Options can be made specific to catalog implementations without weird generic handling, etc. * Using the Registry allows consumers to only import the catalog implementations they care about, allowing any others to be pruned and removed by the compiler as unused and unreachable if not imported. This will enable smaller binaries if not all catalog implementations are needed. -- 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
