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-18243 Reduce scope of collatorsLock spinlock #10401
HPCC-18243 Reduce scope of collatorsLock spinlock #10401
Conversation
https://track.hpccsystems.com/browse/HPCC-18243 |
Moves some logging (that we do see, though possibly should consider reducing frequency of) out of the locked block. |
Automated Smoketest: ✅
Install hpccsystems-platform-community_6.4.2-rc2.el7.x86_64.rpm Unit tests result:
Regression test result:
HPCC Stop: OK |
@ghalliday Please review. Can wait for 6.4.4 or indeed 7.0.0 |
{ | ||
SpinBlock b(collatorsLock); | ||
collators[ruid] = msgColl; | ||
} | ||
msgColl->Link(); |
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.
Is this safe after the SpinBlock?
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.
I think it is, as the object is still owned by the caller.
One question, otherwise looks correct. |
Signed-off-by: Richard Chapman <rchapman@hpccsystems.com>
b4ab955
to
3d6faff
Compare
Signed-off-by: Richard Chapman rchapman@hpccsystems.com
Type of change:
Checklist:
Testing:
TBD