chronon
chronon copied to clipboard
Don't request null keys from KVStore
Summary
Updates the Fetcher to not include or attempt any requests when the keys are null.
Why / Goal
KVStore implementations are left up to consumers. In the case where null entries are being written out, this can lead to online/offline consistency issues. The GroupByUpload jobs are all setup to filter out any null keys, but the KVStore will still attempt to pull values if they exist. This adds parity with the null checks done in the GroupByUpload jobs to the online path and will exclude any attempt to pull values for null keys.
Test Plan
- [x] Added Unit Tests
- [x] Covered by existing CI
- [ ] Integration tested
Checklist
- [ ] Documentation update