Re: [PR] feat(catalog/glue): add support for list namespaces [iceberg-go]

2024-10-15 Thread via GitHub
nastra merged PR #169: URL: https://github.com/apache/iceberg-go/pull/169 -- 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.apa

Re: [PR] feat(catalog/glue): add support for list namespaces [iceberg-go]

2024-10-14 Thread via GitHub
oguzerdogmus commented on code in PR #169: URL: https://github.com/apache/iceberg-go/pull/169#discussion_r1800236271 ## catalog/glue.go: ## @@ -150,8 +151,33 @@ func (c *GlueCatalog) UpdateNamespaceProperties(ctx context.Context, namespace t return PropertiesUpdateSumma

Re: [PR] feat(catalog/glue): add support for list namespaces [iceberg-go]

2024-10-14 Thread via GitHub
zeroshade commented on code in PR #169: URL: https://github.com/apache/iceberg-go/pull/169#discussion_r1800098570 ## catalog/glue.go: ## @@ -150,8 +151,33 @@ func (c *GlueCatalog) UpdateNamespaceProperties(ctx context.Context, namespace t return PropertiesUpdateSummary{

[PR] feat(catalog/glue): add support for list namespaces [iceberg-go]

2024-10-09 Thread via GitHub
oguzerdogmus opened a new pull request, #169: URL: https://github.com/apache/iceberg-go/pull/169 Add support for `ListNamespaces` to the Glue catalog. ### Implementation Details - Similar to tables, used the same `ICEBERG` parameter to differentiate between Iceberg catalog data