| Bug #103788 | Error in PFS_notification_registry's impl | ||
|---|---|---|---|
| Submitted: | 24 May 2021 12:29 | Modified: | 3 Jun 2021 18:07 |
| Reporter: | xiaoyu wang (OCA) | Email Updates: | |
| Status: | Closed | Impact on me: | |
| Category: | MySQL Server: Performance Schema | Severity: | S3 (Non-critical) |
| Version: | MySQL Server 8.0.25 Community | OS: | Any |
| Assigned to: | Marc ALFF | CPU Architecture: | Any |
| Tags: | Contribution, perfschema | ||
[24 May 2021 12:29]
xiaoyu wang
fix it (*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: fix.patch (application/octet-stream, text), 439 bytes.
[24 May 2021 12:39]
MySQL Verification Team
Hello xiaoyu wang, Thank you for the report and contribution. regards, Umesh
[3 Jun 2021 18:07]
Paul DuBois
Posted by developer: Fixed in 8.0.26. Thanks to Xiaoyu Wang for a code correction to PFS_notification_registry::is_empty().
Description: storage/perfschema/service_pfs_notification.cc : L139 bool is_empty() { return m_count > 0; } Shouldn't it be { return m_count == 0 } ? Besides, is_empty() isn't used. How to repeat: Read the source code.