-
Notifications
You must be signed in to change notification settings - Fork 212
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
On cleanup call thd_cleanup function for pool_t objects #2836
Conversation
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 1/405 tests failed ⚠.
The first 10 failing tests are:
ruleset
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 3/405 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
auth
logfill
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 2/405 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
logfill
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: Success ✓.
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 2/405 tests failed ⚠.
The first 10 failing tests are:
sc_timepart_logicalsc_generated
sc_timepart
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.
otherwise changes look good
* cleanup pool_t objects via thd_cleanup: while not a leak, this frees more than 660KB on exit and makes it easier to identify next items to cleanup * break down gbl_thd_linger cond_wait to one second chunks and break/exit as early as possible * tests setup: append test lrl last to take precedence * pthread_attr_setstacksize will fail if size < PTHREAD_STACK_MIN * set gbl_db_is_exiting to timestamp of exiting Signed-off-by: Adi Zaimi <azaimi@bloomberg.net>
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.
LGTM!
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.
Coding style check: Error. Use clang-format or view diff ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 1/406 tests failed ⚠.
The first 10 failing tests are:
deadlock_load_noreorder_generated
and break/exit as early as possible
Signed-off-by: Adi Zaimi azaimi@bloomberg.net