This is an automated email from the ASF dual-hosted git repository.

lihaopeng 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 0a26ad88857 Revert "[Improvement](sort) do not sort partitial when 
spill disabled(#40528)" (#41652)
0a26ad88857 is described below

commit 0a26ad888574962cec3530e45b23455127ecfff6
Author: HappenLee <happen...@hotmail.com>
AuthorDate: Thu Oct 10 19:06:26 2024 +0800

    Revert "[Improvement](sort) do not sort partitial when spill 
disabled(#40528)" (#41652)
    
    Cause mem overflow and use more
---
 be/src/vec/common/sort/sorter.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/be/src/vec/common/sort/sorter.h b/be/src/vec/common/sort/sorter.h
index a290de65bb6..aa7d88dfbc2 100644
--- a/be/src/vec/common/sort/sorter.h
+++ b/be/src/vec/common/sort/sorter.h
@@ -177,8 +177,8 @@ public:
 
 private:
     bool _reach_limit() {
-        return _enable_spill && (_state->unsorted_block_->rows() > 
buffered_block_size_ ||
-                                 _state->unsorted_block_->bytes() > 
buffered_block_bytes_);
+        return _state->unsorted_block_->rows() > buffered_block_size_ ||
+               _state->unsorted_block_->bytes() > buffered_block_bytes_;
     }
 
     Status _do_sort();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to