You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After revisiting ScheduledReporter and its implementations for a bit, have a few ideas for enhancements:
Add callback / promise support to report() that completes when all reporting duties are completed. This will allow the user to detect when reporting completes (so perhaps they can schedule another report if they want to control it manually) or fails.
Using enhancement 1, only allow one in flight report at a time. While it seems unlikely, if there are concurrent reports happening at a time, you could have interlaced data.
Allow CsvReporter and GraphiteReporter to be used without requiring user to call start. This would allow the user to manually control when data is reported with these reporters.
Support duration and rate time unit configurations like dropwizard metrics does so user can request the time unit displayed in their reports.
The text was updated successfully, but these errors were encountered:
After revisiting
ScheduledReporter
and its implementations for a bit, have a few ideas for enhancements:report()
that completes when all reporting duties are completed. This will allow the user to detect when reporting completes (so perhaps they can schedule another report if they want to control it manually) or fails.report
at a time. While it seems unlikely, if there are concurrent reports happening at a time, you could have interlaced data.CsvReporter
andGraphiteReporter
to be used without requiring user to callstart
. This would allow the user to manually control when data is reported with these reporters.The text was updated successfully, but these errors were encountered: