dqhl76 commented on code in PR #2824:
URL:
https://github.com/apache/incubator-opendal/pull/2824#discussion_r1287896325
##########
core/fuzz/utils.rs:
##########
@@ -20,6 +20,19 @@ use std::env;
use opendal::Operator;
use opendal::Scheme;
+pub fn apply_args_to_env() {
+ // for oss-fuzz integration, set envs from args
+ // example: -opendal_fs_test=on -> OPENDAL_FS_TEST=on
Review Comment:
It actually works but it will init fs and mem services at the same time.
Don't we want to split `fuzz_reader` into `fuzz_reader_fs` and
`fuzz_reader_mem`?
If we do that, `fuzz_reader_fs` will init both fs and mem services.
We just do a `cp` operation to split. `fuzz_reader_fs` and
`fuzz_reader_mem` are same.
--
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...@opendal.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
