git-hulk commented on PR #586: URL: https://github.com/apache/incubator-kvrocks/pull/586#issuecomment-1129562831
> > Thanks @PragmaTwice are you familiar with `address sanitizer` and `thread sanitize`. Kvrocks lacks these check on CI currently. We try to do that, but find it doesn't work well. @git-hulk do you remember? > > Maybe a higher version of GCC (>= 5) and `-fsanitize=leak` is needed for detecting memory leaks. BTW I highly recommand Clang while sanitizer or fuzzer instrumentation is required, LeakSanitizer is a part of AddressSanitizer in Clang. Yes, we are running the sanitizer in Clang and can work when memory leak directly in starting the server, but after that. For example, leaks in `HSET` command, the sanitizer can't find out this issue even we ran this command. -- 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]
