You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "tanmayrauth (via GitHub)" <gi...@apache.org> on 2026/04/13 17:29:47 UTC

[PR] fix(table): read snapshot retention properties as defaults in ExpireSnapshots [iceberg-go]

tanmayrauth opened a new pull request, #883:
URL: https://github.com/apache/iceberg-go/pull/883

   
   
   ExpireSnapshots was crashing with "cannot find a valid value" when no caller options were provided. It now falls back to min-snapshots-to-keep, max-snapshot-age-ms, and max-ref-age-ms table properties before using the built-in math.MaxInt defaults, matching Java behavior.
   
   Fixes #839


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [PR] fix(table): read snapshot retention properties as defaults in ExpireSnapshots [iceberg-go]

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade commented on PR #883:
URL: https://github.com/apache/iceberg-go/pull/883#issuecomment-4239088668

   Closing as duplicate of #877 


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [PR] fix(table): read snapshot retention properties as defaults in ExpireSnapshots [iceberg-go]

Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade closed pull request #883: fix(table): read snapshot retention properties as defaults in ExpireSnapshots
URL: https://github.com/apache/iceberg-go/pull/883


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org