PeriodicReader::build make interval registration in same thread/task

[fraillt]

Changes

Previous behavior was that in PeriodicReader::build interval was registered before spawning a task. Which means that we effectively need already running runtime in order to register an interval.

This revision moves interval registration within spawn function, which allow to register PeriodicReader independently of any runtime.

You can see in test collection_triggered_by_interval, that this works perfectly fine without any runtime when using TokioCurrentThread.

Merge requirement checklist

• CONTRIBUTING guidelines followed
• Unit tests added/updated (if applicable)
• Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.7%. Comparing base (3347bde) to head (dcc3b5b).
Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2133   +/-   ##
=====================================
  Coverage   78.7%   78.7%           
=====================================
  Files        121     121           
  Lines      20851   20849    -2     
=====================================
- Hits       16429   16428    -1     
+ Misses      4422    4421    -1     

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

[cijothomas]

LGTM!