yiguolei commented on code in PR #66328:
URL: https://github.com/apache/doris/pull/66328#discussion_r3710084770


##########
be/src/exec/spill/spill_file_writer.cpp:
##########
@@ -149,6 +149,9 @@ Status SpillFileWriter::write_block(RuntimeState* state, 
const Block& block) {
 
     // Lazily open the first part
     if (!_file_writer) {
+        if (_current_part_index == 0) {
+            state->get_query_ctx()->record_spill_data_dir(_data_dir);

Review Comment:
   我们这里为啥需要记录每个data dir啊? 我们是不是只要记录是否存在dir,然后直接全部删除就行了



-- 
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]

Reply via email to