QuakeWang commented on code in PR #41:
URL: https://github.com/apache/paimon-rust/pull/41#discussion_r1703460558
##########
crates/paimon/src/spec/snapshot.rs:
##########
@@ -134,4 +144,38 @@ impl Snapshot {
pub fn statistics(&self) -> Option<&str> {
self.statistics.as_deref()
}
+
+ /// Get the commit time of this snapshot.
+ #[inline]
+ pub fn time_millis(&self) -> u64 {
+ self.time_millis
+ }
+
+ /// Get the commit identifier of this snapshot.
+ #[inline]
+ pub fn commit_identifier(&self) -> i64 {
+ self.commit_identifier
+ }
+
+ /// Get the commit kind of this snapshot.
+ #[inline]
+ pub fn commit_kind(&self) -> CommitKind {
+ self.commit_kind
+ }
+}
+
+/// Type of changes in this snapshot.
+///
+/// Impl Reference:
<https://github.com/apache/paimon/blob/release-0.8.2/paimon-core/src/main/java/org/apache/paimon/Snapshot.java#L506>.
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Deserialize, Serialize)]
+#[serde(rename_all = "camelCase")]
+pub enum CommitKind {
Review Comment:
@Aitozi We do not need this enum, you can refer #11.
--
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...@paimon.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
