Skip to content

Commit

Permalink
chore: allow kernel::visotors to be public for Snapshot usage
Browse files Browse the repository at this point in the history
The Snapshot.files() functrion is public but cannot be possibly used
because the trait it relies upon isn't public. Oops!

Signed-off-by: R. Tyler Croy <[email protected]>
Sponsored-by: Scribd Inc
  • Loading branch information
rtyler committed Jan 13, 2025
1 parent b78952e commit ef066c4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion crates/core/src/kernel/snapshot/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ pub(crate) mod log_segment;
pub(crate) mod parse;
mod replay;
mod serde;
mod visitors;
pub mod visitors;

/// A snapshot of a Delta table
#[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
Expand Down
1 change: 1 addition & 0 deletions crates/core/src/kernel/snapshot/visitors.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ use crate::kernel::Transaction;

/// Allows hooking into the reading of commit files and checkpoints whenever a table is loaded or updated.
pub trait ReplayVisitor: std::fmt::Debug + Send + Sync {
/// Return an [Any] type
fn as_any(&self) -> &dyn std::any::Any;

/// Process a batch
Expand Down

0 comments on commit ef066c4

Please sign in to comment.