Skip to content
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

Fixes #1249 Removed pika log level #2312

Merged

Conversation

muhrin
Copy link
Contributor

@muhrin muhrin commented Dec 6, 2018

This was added before, see #1249, but this no longer happens
as we're using separate a thread for the pika event loop and what's more
the issue has, I believe, been resolved with pika 1.0 which we now use.

This was added before because of issues aiidateam#1249 but this no longer happens
as we're using separate a thread for the pika event loop and what's more
the issue has, I believe, been resolved with pika 1.0 which we now use.
@muhrin muhrin requested a review from sphuber December 6, 2018 18:04
@giovannipizzi giovannipizzi merged commit 818888f into aiidateam:provenance_redesign Dec 6, 2018
@coveralls
Copy link

coveralls commented Dec 8, 2018

Coverage Status

Coverage increased (+0.005%) to 68.936% when pulling ce3d239 on muhrin:fix_1249_pika_log_error into d0a9ad9 on aiidateam:provenance_redesign.

@muhrin muhrin deleted the fix_1249_pika_log_error branch December 12, 2018 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants