1berry/MariaDB-server
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MDEV-34054 Memory leak in Window_func_runner::exec after encountering "temporary space limit reached" error

Browse Source
This commit is contained in:
Monty 2024-05-02 19:07:41 +03:00 committed by Sergei Golubchik
parent 178ab560ae
commit 7d1467e9e9
3 changed files with 29 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
mysql-test/main/tmp_space_usage.result
View File

@ -192,4 +192,11 @@ DROP TABLE t1;
connection default;
disconnect c1;
set @@global.max_tmp_total_space_usage=@save_max_tmp_total_space_usage;
#
# MDEV-34054
# Memory leak in Window_func_runner::exec after encountering
# "temporary space limit reached" error
SET max_tmp_session_space_usage= 64*1024;
SELECT MIN(VARIABLE_VALUE) OVER (), NTILE(1) OVER (), MAX(VARIABLE_NAME) OVER () FROM information_schema.SESSION_STATUS;
ERROR HY000: Local temporary space limit reached
# End of 11.5 tests

9
mysql-test/main/tmp_space_usage.test
View File

@ -250,4 +250,13 @@ connection default;
disconnect c1;
set @@global.max_tmp_total_space_usage=@save_max_tmp_total_space_usage;
--echo #
--echo # MDEV-34054
--echo # Memory leak in Window_func_runner::exec after encountering
--echo # "temporary space limit reached" error
SET max_tmp_session_space_usage= 64*1024;
--error 200
SELECT MIN(VARIABLE_VALUE) OVER (), NTILE(1) OVER (), MAX(VARIABLE_NAME) OVER () FROM information_schema.SESSION_STATUS;
--echo # End of 11.5 tests

17
sql/sql_window.cc
View File

@ -2859,6 +2859,7 @@ bool compute_window_func(THD *thd,
{
List_iterator_fast<Item_window_func> iter_win_funcs(window_functions);
List_iterator_fast<Cursor_manager> iter_cursor_managers(cursor_managers);
bool ret= false;
uint err;
READ_RECORD info;
@ -2921,19 +2922,27 @@ bool compute_window_func(THD *thd,
/* Check if we found any error in the window function while adding values
through cursors. */
if (unlikely(thd->is_error() || thd->is_killed()))
{
ret= true;
break;
}
/* Return to current row after notifying cursors for each window
function. */
if (tbl->file->ha_rnd_pos(tbl->record[0], rowid_buf))
return true;
{
ret= true;
break;
}
}
/* We now have computed values for each window function. They can now
be saved in the current row. */
if (save_window_function_values(window_functions, tbl, rowid_buf))
return true;
{
ret= true;
break;
}
rownum++;
}
@ -2941,7 +2950,7 @@ bool compute_window_func(THD *thd,
partition_trackers.delete_elements();
end_read_record(&info);
return false;
return ret;
}
/* Make a list that is a concation of two lists of ORDER elements */