thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-29 19:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

What if we replace unique_lock with shared?

Browse Source
This commit is contained in:
BoloniniD 2021-07-20 19:22:41 +03:00
parent a9deeb6a78
commit 49ee24e824
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/Storages/StorageFile.cpp
View File

@ -302,6 +302,9 @@ public:
{
if (storage->use_table_fd)
{
shared_lock = std::shared_lock(storage->rwlock, getLockTimeout(context));
if (!shared_lock)
throw Exception("Lock timeout exceeded", ErrorCodes::TIMEOUT_EXCEEDED);
storage->table_fd_was_used = true;
}
else
@ -510,9 +513,10 @@ Pipe StorageFile::read(
{
throw Exception("File descriptor isn't seekable, inside " + this_ptr->getName(), ErrorCodes::CANNOT_SEEK_THROUGH_FILE);
}
ReadBufferFromFileDescriptorPRead table_fd_seek(this_ptr->table_fd);
off_t new_pos = table_fd_seek.seek(this_ptr->table_fd_init_offset, SEEK_SET);
lseek(this_ptr->table_fd, this_ptr->table_fd_init_offset, new_pos);
if (lseek(this_ptr->table_fd, this_ptr->table_fd_init_offset, SEEK_SET) < 0)
{
throw Exception("File descriptor isn't seekable, inside " + this_ptr->getName(), ErrorCodes::CANNOT_SEEK_THROUGH_FILE);
}
}
else
{