unknowntpo commented on PR #5905:
URL: https://github.com/apache/gravitino/pull/5905#issuecomment-2601181334
@diqiu50
This is the new format I came up with:
1. Print filename in separate kv pair, because filename might include `()`,
it's not ok to mix them.
2. Manually print necessary info about `Request`, not rely on `Debug` trait.
3. In reply, we print status first
4. See the example below
```
GETATTR started req.unique=8 filename="test_dir" file_id=10002 uid=501
gid=20 pid=20058 fh=None flags=0
2025-01-18T11:14:55.473292Z DEBUG gvfs_fuse::fuse_api_handle_debug: GETATTR
completed req.unique=8 reply=ttl: 1s, FileAttr: { ino: 10002, size: 0, blocks:
0, atime: "2025-01-18 11:14:55.473280", mtime: "2025-01-18 11:14:55.473280",
ctime: "2025-01-18 11:14:55.473280", crtime: "2025-01-18 11:14:55.473280",
kind: Directory, perm: 700, nlink: 1, uid: 501, gid: 20, rdev: 0, flags: 0,
blksize: 8192 }
...
## For directory listing
2025-01-20T10:00:00.123456Z INFO directory_listing: Directory entry: {
inode: 10001, kind: Directory, name: ".", offset: 1 }
2025-01-20T10:00:00.123457Z INFO directory_listing: Directory entry: {
inode: 1, kind: Directory, name: "..", offset: 2 }
2025-01-20T10:00:00.123458Z INFO directory_listing: Directory entry: {
inode: 10002, kind: RegularFile, name: "test_file", offset: 3 }
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
