zeroshade commented on code in PR #542:
URL: https://github.com/apache/iceberg-go/pull/542#discussion_r2291779078
##########
catalog/glue/glue.go:
##########
@@ -202,16 +202,12 @@ func (c *Catalog) ListTables(ctx context.Context,
namespace table.Identifier) it
// LoadTable loads a table from the catalog table details.
//
// The identifier should contain the Glue database name, then Glue table name.
-func (c *Catalog) LoadTable(ctx context.Context, identifier table.Identifier,
props iceberg.Properties) (*table.Table, error) {
+func (c *Catalog) LoadTable(ctx context.Context, identifier table.Identifier)
(*table.Table, error) {
Review Comment:
the properties passed in allow overriding/adding properties to the returned
table which may be necessary in certain circumstances
--
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
