diqiu50 commented on code in PR #5905:
URL: https://github.com/apache/gravitino/pull/5905#discussion_r1899321481
##########
clients/filesystem-fuse/src/fuse_api_handle.rs:
##########
@@ -30,10 +30,474 @@ use fuse3::{Errno, FileType, Inode, SetAttr, Timestamp};
use futures_util::stream;
use futures_util::stream::BoxStream;
use futures_util::StreamExt;
+use log::debug;
use std::ffi::{OsStr, OsString};
+use std::fmt;
use std::num::NonZeroU32;
use std::time::{Duration, SystemTime};
+/// Wrapper Struct for `Timestamp` to enable custom Display implementation
+pub struct TimestampDebug(pub Timestamp);
+
+impl fmt::Display for TimestampDebug {
+ fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+ let ts = &self.0; // Access the inner `Timestamp`
+ write!(f, "{}.{:09}", ts.sec, ts.nsec) // Nanoseconds padded to 9
digits
+ }
+}
+
+// Optional Debug implementation for `TimestampDebug`
+impl fmt::Debug for TimestampDebug {
+ fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+ write!(f, "Timestamp({})", self) // Reuses `Display` formatting
+ }
+}
+
+pub struct FileAttrDebug<'a> {
+ pub file_attr: &'a FileAttr,
+}
+
+impl<'a> std::fmt::Debug for FileAttrDebug<'a> {
+ fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+ let attr = &self.file_attr;
+ let mut struc = f.debug_struct("FileAttr");
+
+ struc
+ .field("ino", &attr.ino)
+ .field("size", &attr.size)
+ .field("blocks", &attr.blocks)
+ .field("atime", &TimestampDebug(attr.atime))
+ .field("mtime", &TimestampDebug(attr.mtime))
+ .field("ctime", &TimestampDebug(attr.ctime));
+
+ // Conditionally add the "crtime" field only for macOS
+ #[cfg(target_os = "macos")]
+ {
+ struc.field("crtime", &TimestampDebug(attr.crtime));
+ }
+
+ struc
+ .field("kind", &attr.kind)
+ .field("perm", &attr.perm)
+ .field("nlink", &attr.nlink)
+ .field("uid", &attr.uid)
+ .field("gid", &attr.gid)
+ .field("rdev", &attr.rdev)
+ .finish()
+ }
+}
+
+pub(crate) struct FuseApiHandleDebug<T: RawFileSystem> {
+ inner: FuseApiHandle<T>,
+}
+
+impl<T: RawFileSystem> FuseApiHandleDebug<T> {
+ pub fn new(fs: T, context: FileSystemContext) -> Self {
+ Self {
+ inner: FuseApiHandle::new(fs, context),
+ }
+ }
+}
+
+impl<T: RawFileSystem> Filesystem for FuseApiHandleDebug<T> {
+ async fn init(&self, req: Request) -> fuse3::Result<ReplyInit> {
+ debug!("init [id={}]: req: {:?}", req.unique, req);
+ let res = self.inner.init(req).await;
+ match res {
+ Ok(reply) => {
+ debug!("init [id={}]: reply: {:?}", req.unique, reply);
+ Ok(reply)
+ }
+ Err(e) => {
+ debug!("init [id={}]: error: {:?}", req.unique, e);
+ Err(e)
Review Comment:
Is using `error!` better?
--
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: commits-unsubscr...@gravitino.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
