refactor: rename Snapshot.checkpoint (delta-io#1608)

## 🥞 Stacked PR
Use this
[link](https://github.com/delta-io/delta-kernel-rs/pull/1608/files) to
review incremental changes.
-
[**stack/rename-checkpoint**](delta-io#1608)
[[Files
changed](https://github.com/delta-io/delta-kernel-rs/pull/1608/files)]
-
[stack/perform-checkpoint](delta-io#1600)
[[Files
changed](https://github.com/delta-io/delta-kernel-rs/pull/1600/files/b2556bc302d7fe25cdb879f3fc085075c6835240..743587c71cddca53a7313b4d515d0ad1495748e4)]
-
[stack/ffi-checkpoint](delta-io#1611)
[[Files
changed](https://github.com/delta-io/delta-kernel-rs/pull/1611/files/743587c71cddca53a7313b4d515d0ad1495748e4..6d390888a36ce2867b1375b49b7976145ceb27a0)]

---------
## What changes are proposed in this pull request?
Current `snapshot.checkpoint()` method actually creates a
`CheckpointWriter`. This PR renamed it to `create_checkpoint_writer` so
that the name better aligns with the functionality.
<!--
**Uncomment** this section if there are any changes affecting public
APIs. Else, **delete** this section.

### This PR affects the following public APIs

If there are breaking changes, please ensure the `breaking-changes`
label gets added by CI, and describe why the changes are needed.

Note that _new_ public APIs are not considered breaking.
-->

## How was this change tested?

Existing.

Author: dengsh12

kernel/examples/checkpoint-table/src/main.rs

@@ -68,7 +68,7 @@ async fn try_main() -> DeltaResult<()> {
     let snapshot = Snapshot::builder_for(url).build(&engine)?;
 
     // first we create a checkpoint writer
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
 
     // this tells us the path where we should write the checkpoint file
     let checkpoint_path = writer.checkpoint_path()?;

kernel/src/checkpoint/mod.rs

@@ -1,6 +1,6 @@
 //! This module implements the API for writing single-file checkpoints.
 //!
-//! The entry point for this API is [`Snapshot::checkpoint`].
+//! The entry point for this API is [`Snapshot::create_checkpoint_writer`].
 //!
 //! ## Checkpoint Types and Selection Logic
 //! This API supports two checkpoint types, selected based on table features:
@@ -22,7 +22,7 @@
 //!
 //! The following steps outline the process of creating a checkpoint:
 //!
-//! 1. Create a [`CheckpointWriter`] using [`Snapshot::checkpoint`]
+//! 1. Create a [`CheckpointWriter`] using [`Snapshot::create_checkpoint_writer`]
 //! 2. Get the checkpoint path from [`CheckpointWriter::checkpoint_path`]
 //! 2. Get the checkpoint data from [`CheckpointWriter::checkpoint_data`]
 //! 3. Write the data to the path in object storage (engine-specific)
@@ -51,7 +51,7 @@
 //! let snapshot = Snapshot::builder_for(url).build(engine)?;
 //!
 //! // Create a checkpoint writer from the snapshot
-//! let mut writer = snapshot.checkpoint()?;
+//! let mut writer = snapshot.create_checkpoint_writer()?;
 //!
 //! // Get the checkpoint path and data
 //! let checkpoint_path = writer.checkpoint_path()?;
@@ -80,7 +80,7 @@
 //!
 //! [`CheckpointMetadata`]: crate::actions::CheckpointMetadata
 //! [`LastCheckpointHint`]: crate::last_checkpoint_hint::LastCheckpointHint
-//! [`Snapshot::checkpoint`]: crate::Snapshot::checkpoint
+//! [`Snapshot::create_checkpoint_writer`]: crate::Snapshot::create_checkpoint_writer
 // Future extensions:
 // - TODO(#837): Multi-file V2 checkpoints are not supported yet. The API is designed to be extensible for future
 //   multi-file support, but the current implementation only supports single-file checkpoints.

kernel/src/checkpoint/tests.rs

@@ -75,7 +75,7 @@ async fn test_create_checkpoint_metadata_batch() -> DeltaResult<()> {
 
     let table_root = Url::parse("memory:///")?;
     let snapshot = Snapshot::builder_for(table_root).build(&engine)?;
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
 
     let checkpoint_batch = writer.create_checkpoint_metadata_batch(&engine)?;
     assert!(checkpoint_batch.filtered_data.has_selected_rows());
@@ -296,7 +296,7 @@ async fn test_v1_checkpoint_latest_version_by_default() -> DeltaResult<()> {
 
     let table_root = Url::parse("memory:///")?;
     let snapshot = Snapshot::builder_for(table_root).build(&engine)?;
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
 
     // Verify the checkpoint file path is the latest version by default.
     assert_eq!(
@@ -368,7 +368,7 @@ async fn test_v1_checkpoint_specific_version() -> DeltaResult<()> {
     let snapshot = Snapshot::builder_for(table_root)
         .at_version(0)
         .build(&engine)?;
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
 
     // Verify the checkpoint file path is the specified version.
     assert_eq!(
@@ -419,7 +419,7 @@ async fn test_finalize_errors_if_checkpoint_data_iterator_is_not_exhausted() ->
     let snapshot = Snapshot::builder_for(table_root)
         .at_version(0)
         .build(&engine)?;
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
     let data_iter = writer.checkpoint_data(&engine)?;
 
     /* The returned data iterator has batches that we do not consume */
@@ -475,7 +475,7 @@ async fn test_v2_checkpoint_supported_table() -> DeltaResult<()> {
 
     let table_root = Url::parse("memory:///")?;
     let snapshot = Snapshot::builder_for(table_root).build(&engine)?;
-    let writer = snapshot.checkpoint()?;
+    let writer = snapshot.create_checkpoint_writer()?;
 
     // Verify the checkpoint file path is the latest version by default.
     assert_eq!(
@@ -556,7 +556,7 @@ async fn test_no_checkpoint_staged_commits() -> DeltaResult<()> {
         .build(&engine)?;
 
     assert!(matches!(
-        snapshot.checkpoint().unwrap_err(),
+        snapshot.create_checkpoint_writer().unwrap_err(),
         crate::Error::Generic(e) if e == "Found staged commit file in log segment"
     ));
     Ok(())

kernel/src/snapshot.rs

@@ -380,7 +380,7 @@ impl Snapshot {
     ///
     /// See the [`crate::checkpoint`] module documentation for more details on checkpoint types
     /// and the overall checkpoint process.
-    pub fn checkpoint(self: Arc<Self>) -> DeltaResult<CheckpointWriter> {
+    pub fn create_checkpoint_writer(self: Arc<Self>) -> DeltaResult<CheckpointWriter> {
         CheckpointWriter::try_new(self)
     }