database/mariadb-columnstore-engine
1
0
Fork 0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-07-30 19:23:07 +03:00
Code Activity

fix(join, disk-based): MCOL-5597: large side read errors (#3117)

Browse Source 
The large side read errors mentioned there can be due to failure to
close file stream properly. Some of the data may still reside in the
file stream buffers, closing must flush it. The flush is an I/O
operation and can fail, leading to partial write and subsequent partial
read.

This patch tries to provide better diagnostics.
This commit is contained in:
Sergey Zefirov
2024-02-09 22:25:43 +03:00
committed by GitHub
parent 8de4eec362
commit ebcf43a517
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
utils/joiner/joinpartition.cpp
View File

@ -849,7 +849,15 @@ uint64_t JoinPartition::writeByteStream(int which, ByteStream& bs)
bs.advance(len);
offset = fs.tellp();
fs.close();
if (fs.fail())
{
ostringstream os;
os << "Disk join file " << filename << ": close() failure, probable exhaustion of disk space." << endl;
throw IDBExcept(os.str().c_str(), ERR_DBJ_FILE_IO_ERROR);
}
return ret;
}