unknowntpo commented on code in PR #5905:
URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580732
##########
clients/filesystem-fuse/src/main.rs:
##########
@@ -193,6 +213,33 @@ fn main() -> Result<(), i32> {
path.to_string_lossy().to_string()
};
+ if env::var("RUST_LOG").is_ok() {
+ init_tracing_subscriber(LevelFilter::INFO, "");
+ } else {
+ // if debug > 0, it means that we needs fuse_debug.
+ app_config.fuse.fuse_debug = debug > 0 ||
app_config.fuse.fuse_debug;
+ match debug {
+ 0 => {
+ init_tracing_subscriber(LevelFilter::INFO, "");
Review Comment:
I want to list some examples to make sure my understanding is correct:
1. When `debug` is not set, we use value of `RUST_LOG` as directives, the
actual effect should be RUST_LOG="<original_rust_log_value>".
2. when `debug != 0`, we particularly we need to get the
`gvfs_fuse::fuse_api_handle_debug=debug` and append to the value of `RUST_LOG`,
the actual effect should be
`RUST_LOG="<original_rust_log_value>,gvfs_fuse::fuse_api_handle_debug=debug"`.
This make sure that no matter which level is set by `RUST_LOG` for `gvfs_fuse`,
`gvfs_fuse::fuse_api_handle_debug` still has debug level.
--
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
