-
Notifications
You must be signed in to change notification settings - Fork 301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HPCC-32397 Ensure delta writer is stopped earlier to avoid crash #19176
HPCC-32397 Ensure delta writer is stopped earlier to avoid crash #19176
Conversation
The delta writer must be stopped during closedown, before other dependent members of the SDS manager are destroyed. Signed-off-by: Jake Smith <jake.smith@lexisnexisrisk.com>
Jira Issue: https://hpccsystems.atlassian.net//browse/HPCC-32397 Jirabot Action Result: |
@@ -2225,7 +2225,7 @@ void CDeltaWriter::addToQueue(CTransactionItem *item) | |||
++totalQueueLimitHits; | |||
// force a synchronous save | |||
CCycleTimer timer; | |||
PROGLOG("Forcing synchronous save of %u transactions", (unsigned)pending.size()); | |||
PROGLOG("Forcing synchronous save of %u transactions (pendingSz=%zu)", (unsigned)pending.size(), pendingSz); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
totally incidental change - feel free to reject for it to go into a separate PR
@@ -6056,6 +6056,7 @@ CCovenSDSManager::~CCovenSDSManager() | |||
{ | |||
if (unhandledThread) unhandledThread->join(); | |||
if (coalesce) coalesce->stop(); | |||
deltaWriter.stop(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NB: before this change, stop() would happen via member dtor.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ghalliday - please review.
Wasn't sure whether to target 9.6 or 9.8 - decided on 9.8 and pondered whether should go to master, because it is long term bug, that affects shutdown only.
I think it is fair to merge fixes for crashes and cores into the current release as long as they are relatively safe |
The delta writer must be stopped during closedown, before other dependent members of the SDS manager are destroyed.
Type of change:
Checklist:
Smoketest:
Testing: