ModelarData / ModelarDB-RS

ModelarDB: Model-Based Time Series Management from Edge to Cloud
Apache License 2.0
11 stars 5 forks source link

Fix #187 by storing univariate_id as int64 #198

Closed skejserjensen closed 1 month ago

skejserjensen commented 1 month ago

This PR fixes #187 by storing univariate_id as Int64Array instead of UInt64Array as the Delta Lake Protocol does not support unsigned integers and #197 probably will not be resolved for some time. When reviewing this PR, please double check the necessary casts are performed everywhere they are needed. Currently, UInt64Array is reinterpreted as Int64Array when writing compressed segments to disk and Int64Array is reinterpreted as UInt64Array when data points are reconstructed from segments and aggregates are computed directly from the segments.