feat(tianmu): remove DBUG_OFF and repalce DEBUG_ASSERT with assert

[duanjr]

Summary about this PR

feat: remove DBUG_OFF flag and repalce DEBUG_ASSERT with assert in tianmu engine(#1551)

1.Remove the code segment in common/assert.h

#ifdef DBUG_OFF
#define DEBUG_ASSERT(condition) \
  do {                          \
  } while (false)
#else
#define DEBUG_ASSERT(...) assert(__VA_ARGS__)
#endif

2.Automatically replace DBUG_ASSERT with assert in all files under tianmu directory.

3.Manually adjust indent in core/query.cpp and optimizer/compile/descriptor.cpp

[Edited]4.Automatically update fomat use clang-format after first check failed.

Tests Check List

• Unit test
• Integration test
• Manual test (add detailed scripts or steps below)
• No code

Changelog

• New Feature
• Bug Fix
• Performance Improvement
• Build/Testing/CI/CD
• Documentation
• Not for changelog (changelog entry is not required)

Documentation

• Affects user behaviors
• Contains syntax changes
• Contains variable changes
• Contains experimental features

[mergify]

This pull request's title should follow requirements next. @duanjr please check it 👇.

Valid format:

fix(vc): fix sth..... (#3306)
  ^         ^---------^  ^----^
  |         |            |
  |         +            +-> you issue id.
  |         |
  |         +-> Summary in present tense.
  |
  +-------> Type: feat, fix, docs, workflow, style, refactor, test, website, chore

Valid types:

• feat: new feature for stonedb
• fix: bug fix for stonedb
• docs: changes to the documentation
• workflow: ci/cd in .github
• perf: Changes to improve code performance
• refactor: refactoring production code, eg. renaming a variable
• style: formatting, missing semi colons, etc; no production code change
• test: adding missing tests, refactoring tests; no production code change
• website
• chore: updating grunt tasks etc; no production code change

[mergify]

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

[hustjieke]

Greate job!

[codecov]

Codecov Report

Patch coverage: 45.45% and project coverage change: +0.02 🎉

Comparison is base (74b5b05) 55.18% compared to head (7d06683) 55.20%.

❗ Current head 7d06683 differs from pull request most recent head 28647ad. Consider uploading reports for the commit 28647ad to get more accurate results

Additional details and impacted files

@@                 Coverage Diff                 @@
##           stonedb-5.7-dev    #1751      +/-   ##
===================================================
+ Coverage            55.18%   55.20%   +0.02%     
===================================================
  Files                 2031     2031              
  Lines               431436   431424      -12     
===================================================
+ Hits                238081   238181     +100     
+ Misses              193355   193243     -112     

Impacted Files	Coverage Δ
storage/tianmu/compress/basic_data_filt.cpp	45.03% <0.00%> (ø)
storage/tianmu/compress/data_stream.h	62.90% <ø> (ø)
storage/tianmu/compress/dictionary.h	26.61% <0.00%> (ø)
storage/tianmu/compress/inc_alloc.cpp	26.41% <ø> (ø)
storage/tianmu/compress/inc_alloc.h	6.66% <ø> (ø)
storage/tianmu/compress/inc_wgraph.cpp	2.19% <ø> (ø)
storage/tianmu/compress/inc_wgraph.h	5.88% <ø> (ø)
storage/tianmu/compress/ppm.cpp	0.00% <ø> (ø)
storage/tianmu/compress/suffix_tree.cpp	0.00% <0.00%> (ø)
storage/tianmu/compress/text_compressor.cpp	13.33% <ø> (ø)
... and 125 more

... and 47 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[hustjieke]

LGTM

[hustjieke]

LGTM