filodb / FiloDB

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

misc(iceberg-export): remove assert statement as some labels may not exist in all timeseries #1743

Closed nikitag55 closed 7 months ago

nikitag55 commented 7 months ago

Current behavior :

Some labels as configured in label-column-mapping (to add as more columns in the table) may not exist in all the timeseries. So, the assert statement can fail the export job if the label may not exist in the timeseries and hence, label-value can be none.

New behavior :

Remove the assert statement such that, if the label doesn't exist in the timeseries, store its value as none in its table column.

Remove assert and handle key not found in Map gracefully.