chronon icon indicating copy to clipboard operation
chronon copied to clipboard

[Fetcher] Print exceptions prior to reporting for metrics

Open nikhilsimha opened this issue 1 year ago • 4 comments

Summary

We now expose exceptionSampleRate via API - default is 0.1 - one among every 10 exceptions will be logged.

Why / Goal

Currently we don't log exceptions at all, which makes debugging difficult. This will make it so that by default we print a fraction of exceptions.

Test Plan

  • [ ] Added Unit Tests
  • [x] Covered by existing CI
  • [ ] Integration tested

Checklist

  • [ ] Documentation update

Reviewers

@hzding621 @qiyang0221

nikhilsimha avatar Oct 06 '23 20:10 nikhilsimha