github-actions[bot] commented on code in PR #64866:
URL: https://github.com/apache/doris/pull/64866#discussion_r3490260673
##########
be/test/exec/runtime_filter/runtime_filter_merger_test.cpp:
##########
@@ -66,12 +69,14 @@ class RuntimeFilterMergerTest : public RuntimeFilterTest {
merger->set_expected_producer_num(1);
Review Comment:
This test file still calls the old
`RuntimeFilterMerger::set_expected_producer_num(...)` helper, but the PR
removed that member and replaced the production users with
`increase_expected_producer_num(...)`. The only remaining
`set_expected_producer_num` references are the four calls in this file, so
`RuntimeFilterMergerTest` will fail to compile once the BE UT target is built.
Please update these calls to the new helper name, or keep a compatibility
wrapper if the old API is still intended.
--
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]
