[fix](be) Add explicit template instantiation for BlockFileCache::get_cell#64228
Open
heguanhui wants to merge 1 commit into
Open
[fix](be) Add explicit template instantiation for BlockFileCache::get_cell#64228heguanhui wants to merge 1 commit into
heguanhui wants to merge 1 commit into
Conversation
Contributor
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
2 tasks
Contributor
Author
|
run buildall |
Contributor
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
Contributor
TPC-H: Total hot run time: 29341 ms |
Contributor
TPC-DS: Total hot run time: 169194 ms |
5c59427 to
b51f3a0
Compare
Contributor
Author
|
run buildall |
Contributor
TPC-H: Total hot run time: 29056 ms |
Contributor
TPC-DS: Total hot run time: 169804 ms |
Contributor
Author
|
run buildall |
b51f3a0 to
17e274c
Compare
Contributor
Author
|
run buildall |
Contributor
TPC-H: Total hot run time: 28923 ms |
Contributor
TPC-DS: Total hot run time: 168950 ms |
17e274c to
83f03d5
Compare
Contributor
Author
|
run buildall |
Contributor
TPC-H: Total hot run time: 28878 ms |
Contributor
TPC-DS: Total hot run time: 177416 ms |
Contributor
ClickBench: Total hot run time: 25.34 s |
Contributor
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
Contributor
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
Contributor
Author
|
run nonConcurrent |
83f03d5 to
f96d3cb
Compare
Contributor
Author
|
run buildall |
Contributor
TPC-H: Total hot run time: 28854 ms |
Contributor
TPC-DS: Total hot run time: 175285 ms |
Contributor
ClickBench: Total hot run time: 25.19 s |
Contributor
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
Issue Number: close #64227
Related PR: #xxx
Problem Summary: When building BE UT with TSAN, the linker reports undefined symbol for
BlockFileCache::get_cell<std::lock_guard<std::mutex>>. The template functionget_cellis defined inblock_file_cache.cppbut lacks explicit template instantiation. Non-TSAN builds may inline the template, masking the issue, but TSAN disables inlining, exposing the missing symbol.normal log after fix:
tsan_get_cell_link_error_TSAN.log
The same pattern already exists for the
removetemplate function (block_file_cache.cpp:2515) andLRUQueuetemplates (file_cache_common.cpp:149-150).get_cellis simply missing the same explicit instantiation.Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)