apache / tsfile

Apache TsFile
https://tsfile.apache.org/
Apache License 2.0
104 stars 50 forks source link

Enable address sanitizer in tsfile cpp. #214

Closed ColinLeeo closed 2 months ago

ColinLeeo commented 3 months ago

In this pr:

  1. Fixed the issue where a flush operation would result in an E_ALLREAY_EXIST error after inserting a large amount of data.
  2. Enabled Address Sanitizer in tests.
  3. Checked and fixed memory leaks.
  4. Removed some unused configuration items.
  5. Performed code formatting and cleanup.
ColinLeeo commented 3 months ago

It seems that compiling tsfile_cpp with asan at Windows needs clang instead of Mingw.

jt2594838 commented 3 months ago

It seems that compiling tsfile_cpp with asan at Windows needs clang instead of Mingw.

Yeah, how about only enabling it on linux. I think it enough for detecting memory leaks.