This is an automated email from the ASF dual-hosted git repository.
gavinchou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 5b59b2f1b78 [enhancement](cloud) unify scanner thread pool as LOCAL in
cloud mode (#38372)
5b59b2f1b78 is described below
commit 5b59b2f1b78e4269eae1979d900f25a710950f8f
Author: zhengyu <[email protected]>
AuthorDate: Thu Jul 25 21:36:59 2024 +0800
[enhancement](cloud) unify scanner thread pool as LOCAL in cloud mode
(#38372)
Current scanner is spread in both Scan & RScan thread pool. Unify them
to get better performance (+10% in TPCH1T & SSB1T bench).
Before, two thread pools are both working:
After, only local pool is working
Signed-off-by: freemandealer <[email protected]>
---
be/src/vec/exec/scan/new_olap_scanner.cpp | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/be/src/vec/exec/scan/new_olap_scanner.cpp
b/be/src/vec/exec/scan/new_olap_scanner.cpp
index 36e1c3d41a5..ae785d92b16 100644
--- a/be/src/vec/exec/scan/new_olap_scanner.cpp
+++ b/be/src/vec/exec/scan/new_olap_scanner.cpp
@@ -499,6 +499,10 @@ Status NewOlapScanner::_init_return_columns() {
}
doris::TabletStorageType NewOlapScanner::get_storage_type() {
+ if (config::is_cloud_mode()) {
+ // we don't have cold storage in cloud mode, all storage is treated as
local
+ return doris::TabletStorageType::STORAGE_TYPE_LOCAL;
+ }
int local_reader = 0;
for (const auto& reader : _tablet_reader_params.rs_splits) {
local_reader += reader.rs_reader->rowset()->is_local();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
