bitsondatadev commented on code in PR #9591: URL: https://github.com/apache/iceberg/pull/9591#discussion_r1475087807
########## docs/docs/branching.md: ########## @@ -33,14 +25,14 @@ menu: Iceberg table metadata maintains a snapshot log, which represents the changes applied to a table. Snapshots are fundamental in Iceberg as they are the basis for reader isolation and time travel queries. -For controlling metadata size and storage costs, Iceberg provides snapshot lifecycle management procedures such as [`expire_snapshots`](../spark-procedures/#expire-snapshots) for removing unused snapshots and no longer necessary data files based on table snapshot retention properties. +For controlling metadata size and storage costs, Iceberg provides snapshot lifecycle management procedures such as [`expire_snapshots`](spark-procedures.md#expire-snapshots) for removing unused snapshots and no longer necessary data files based on table snapshot retention properties. Review Comment: so the plugin doesn't have anything out of the box but I was wrong actually, the current link fails for the current site: https://iceberg.apache.org/docs/latest/spark-procedures/#expire-snapshots The reason my linkchecker missed this was that it still loads the page just not the link. So we'll need to do a pass at these in a later PR. -- 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
