Mryange opened a new pull request, #70360: URL: https://github.com/apache/doris/pull/70360
### Background BE enables strict Clang warnings, with local diagnostic pragmas for exceptions. ### What problem does this PR solve? Issue Number: None Related PR: None PR Type: Refactor Problem Summary: Rename `NGramBloomFilter::_size` to `_filter_size` to avoid hiding the base member. Remove obsolete suppressions from `PipelineFragmentContext` and `compaction.h`, and remove the unbalanced, file-wide shadow warning suppression from `vmatch_predicate.cpp`. ### Details - **Moves vs. logic changes:** Member rename and diagnostic cleanup only. - **Behavior change:** None. - **Compatibility:** No storage format or protocol changes. - **Performance impact:** None expected. - **Rollback:** Revert this commit. ### Testing - Clang ASAN build: `./build.sh --be -j48` passed. - clang-format 16, `git diff --check`, and BE build hygiene checks passed. - Repository clang-tidy checks passed for changed lines (Clang resource directory supplied explicitly). - Unit and regression tests were not run; runtime behavior is unchanged. ### Release note None. ### Check List (For Author) - Behavior changed: No - Does this need documentation: No - Pre-submission status: Ready ### AI Disclosure - Tool: OpenAI Codex - Generated: Code changes and PR description. - Human review: Pending author review. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
