This is an automated email from the ASF dual-hosted git repository.
zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-go.git
The following commit(s) were added to refs/heads/main by this push:
new 8d7bfa835 refactor(catalog)!: remove deprecated Closer alias before v1
(#2332)
8d7bfa835 is described below
commit 8d7bfa8356b2629e0d63de6349e8f5e6d506eb7e
Author: Minh Vu <[email protected]>
AuthorDate: Mon Oct 26 20:48:17 2026 +0100
refactor(catalog)!: remove deprecated Closer alias before v1 (#2332)
* refactor(catalog)!: remove deprecated Closer alias
Signed-off-by: Minh Vu <[email protected]>
* docs(catalog): show migration from Closer assertion
Signed-off-by: Minh Vu <[email protected]>
---------
Signed-off-by: Minh Vu <[email protected]>
Co-authored-by: Matt Topol <[email protected]>
---
catalog/catalog.go | 5 -----
website/src/releases.md | 2 +-
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/catalog/catalog.go b/catalog/catalog.go
index f851e800b..ff64071a8 100644
--- a/catalog/catalog.go
+++ b/catalog/catalog.go
@@ -162,11 +162,6 @@ type Catalog interface {
removals []string, updates iceberg.Properties)
(PropertiesUpdateSummary, error)
}
-// Closer is retained as an alias for io.Closer.
-//
-// Deprecated: Use [io.Closer] instead; Catalog embeds it directly.
-type Closer = io.Closer
-
// TransactionalCatalog is an optional interface implemented by catalogs
// that support atomic multi-table commits. Callers should check for this
// capability via a type assertion:
diff --git a/website/src/releases.md b/website/src/releases.md
index 409c8646f..636056764 100644
--- a/website/src/releases.md
+++ b/website/src/releases.md
@@ -52,7 +52,7 @@ Per-release notes (highlights, breaking changes,
contributors) are published on
- **Substrait expression conversion no longer takes case sensitivity.**
`substrait.ConvertExpr(schema, expression, caseSensitive)` becomes
`substrait.ConvertExpr(schema, expression)`. Case sensitivity remains part of
expression binding.
- **Transform formatting requires the result type and uses the Java-aligned
method name.** `Transform.ToHumanStr(value)` and
`Transform.ToHumanStrType(resultType, value)` are removed. Use
`Transform.ToHumanString(resultType, value)` to format partition values,
including timestamps with time zones.
-- **Catalogs now require cleanup.** The `catalog.Catalog` interface includes
`Close() error`. Custom catalog implementations must provide `Close`, and
callers should close catalog instances after use. `catalog.Closer` remains
available as an alias for `io.Closer`.
+- **Catalogs now require cleanup.** The `catalog.Catalog` interface includes
`Close() error`. Custom catalog implementations must provide `Close`, and
callers should close catalog instances after use. `catalog.Closer` is removed.
Replace `cat.(catalog.Closer)` checks with `defer cat.Close()`;
`catalog.Catalog` already implements `io.Closer`, so no type assertion is
needed.
- **Puffin writer property setter renamed.** `puffin.Writer.AddProperties` is
now `puffin.Writer.SetProperties`. Repeated calls still merge keys; use
`ClearProperties` to clear custom properties before finishing.
