Attempt at cleaning up getting the next bucket

[jeltz]

The code for getting the next bucket was unnecessarily complex and hard to read. But is this suggestion good enough? Maybe we need to do an even bigger rewrite of the bucketing? Especially since the whole mess with two atomics and taking an exclusive lock here is pretty odd and unclear if it is even correct for example if we skip some buckets. Will they get cleared out then?

PG-0

[codecov]

Codecov Report

❌ Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.09%. Comparing base (479f326) to head (6d86675).

Files with missing lines	Patch %	Lines
src/pg_stat_monitor.c	94.44%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #723      +/-   ##
==========================================
+ Coverage   87.89%   88.09%   +0.20%     
==========================================
  Files           3        3              
  Lines        1280     1277       -3     
  Branches      202      201       -1     
==========================================
  Hits         1125     1125              
+ Misses         69       67       -2     
+ Partials       86       85       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.