You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@polaris.apache.org by Yufei Gu <fl...@gmail.com> on 2025/03/31 18:54:08 UTC

Rename table maintenance policy "snapshot-retention" to "snapshot-expiry"

Hi folks,

I'm proposing to rename the table maintenance policy "snapshot-retention"
to "snapshot-expiry". While "snapshot-retention" is a reasonable name, it's
a bit too general. I believe "snapshot-expiry" more precisely captures the
intent of the policy, which is specifically about removing expired
snapshots.

I think this is the right time to make the change before we include the
policy schemas in an upcoming release.

Here’s the PR for reference: https://github.com/apache/polaris/pull/1284

If you have any concerns or suggestions for a better name, feel free to
share your thoughts.

Yufei

Re: Rename table maintenance policy "snapshot-retention" to "snapshot-expiry"

Posted by Dmitri Bourlatchkov <di...@apache.org>.
For visibility: I posed my comment in GH (minor):
https://github.com/apache/polaris/pull/1284#discussion_r2022896891

Cheers,
Dmitri.

On Mon, Mar 31, 2025 at 2:55 PM Yufei Gu <fl...@gmail.com> wrote:

> Hi folks,
>
> I'm proposing to rename the table maintenance policy "snapshot-retention"
> to "snapshot-expiry". While "snapshot-retention" is a reasonable name, it's
> a bit too general. I believe "snapshot-expiry" more precisely captures the
> intent of the policy, which is specifically about removing expired
> snapshots.
>
> I think this is the right time to make the change before we include the
> policy schemas in an upcoming release.
>
> Here’s the PR for reference: https://github.com/apache/polaris/pull/1284
>
> If you have any concerns or suggestions for a better name, feel free to
> share your thoughts.
>
> Yufei
>

Re: Rename table maintenance policy "snapshot-retention" to "snapshot-expiry"

Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.
+1

It makes sense to me.

Regards
JB

On Mon, Mar 31, 2025 at 8:54 PM Yufei Gu <fl...@gmail.com> wrote:
>
> Hi folks,
>
> I'm proposing to rename the table maintenance policy "snapshot-retention"
> to "snapshot-expiry". While "snapshot-retention" is a reasonable name, it's
> a bit too general. I believe "snapshot-expiry" more precisely captures the
> intent of the policy, which is specifically about removing expired
> snapshots.
>
> I think this is the right time to make the change before we include the
> policy schemas in an upcoming release.
>
> Here’s the PR for reference: https://github.com/apache/polaris/pull/1284
>
> If you have any concerns or suggestions for a better name, feel free to
> share your thoughts.
>
> Yufei