You've already forked mariadb-columnstore-engine
mirror of
https://github.com/mariadb-corporation/mariadb-columnstore-engine.git
synced 2025-08-08 14:22:09 +03:00
Fix some error case handling to not update counts or write metadata when not required.
This commit is contained in:
@@ -422,7 +422,6 @@ ssize_t IOCoordinator::_write(const boost::filesystem::path &filename, const uin
|
|||||||
if (err < 0)
|
if (err < 0)
|
||||||
{
|
{
|
||||||
l_errno = errno;
|
l_errno = errno;
|
||||||
replicator->updateMetadata(metadata);
|
|
||||||
//log error and abort
|
//log error and abort
|
||||||
logger->log(LOG_ERR,"IOCoordinator::write(): Failed addJournalEntry -- NullData. Journal file likely has partially written data and incorrect metadata.");
|
logger->log(LOG_ERR,"IOCoordinator::write(): Failed addJournalEntry -- NullData. Journal file likely has partially written data and incorrect metadata.");
|
||||||
errno = l_errno;
|
errno = l_errno;
|
||||||
@@ -430,9 +429,6 @@ ssize_t IOCoordinator::_write(const boost::filesystem::path &filename, const uin
|
|||||||
}
|
}
|
||||||
else if ((uint)err < nullJournalSize)
|
else if ((uint)err < nullJournalSize)
|
||||||
{
|
{
|
||||||
dataRemaining -= err;
|
|
||||||
count += err;
|
|
||||||
iocBytesWritten += err;
|
|
||||||
if ((err + lastObject.length) > lastObject.length)
|
if ((err + lastObject.length) > lastObject.length)
|
||||||
metadata.updateEntryLength(lastObject.offset, (err + lastObject.length));
|
metadata.updateEntryLength(lastObject.offset, (err + lastObject.length));
|
||||||
cache->newJournalEntry(firstDir, err+JOURNAL_ENTRY_HEADER_SIZE);
|
cache->newJournalEntry(firstDir, err+JOURNAL_ENTRY_HEADER_SIZE);
|
||||||
|
Reference in New Issue
Block a user