filodb / FiloDB

Distributed Prometheus time series database
Apache License 2.0
1.43k stars 225 forks source link

fix bugs caused by stitching empty and non-empty data. #1832

Closed yu-shipit closed 2 months ago

yu-shipit commented 2 months ago

The current Transient row can only handle a schema (Long, Double). However, there are different schemas for histogram, Avg and so on. Create a NaNRowReader to handle any schemas.

Pull Request checklist

Current behavior : (link exiting issues here : https://help.github.com/articles/basic-writing-and-formatting-syntax/#referencing-issues-and-pull-requests)

New behavior :

BREAKING CHANGES

If this PR contains a breaking change, please describe the impact and migration path for existing applications. If not please remove this section.

Breaking changes may include:

Other information:

alextheimer commented 2 months ago

@yu-shipit Could you also add a brief description to the PR header?