Stats agg #141

davidkohn88 · 2021-05-25T15:33:39Z

No description provided.

davidkohn88 · 2021-05-25T15:43:49Z

docs/rolling_average_api_working.md

+```SQL
+moving_average(window => '30 minutes', slide => '5 minutes', data)
+```
+However, because set-returning aggregates cannot not exist in Postgres, this will not work outside of the timeseries API. Currently, doing rolling average properly requires windowed aggregates. In base SQL it is a real PITA because you have to do sum and count separately and then divide them yourself.


We should probably also think about the continuous aggregate implications ie #138 in this document.

crates/counter-agg/src/lib.rs

crates/stats-agg/src/stats1d.rs

WireBaron · 2021-06-17T17:27:17Z

extension/src/stats_agg.rs

+
+#[pg_extern(schema = "timescale_analytics_experimental")]
+pub fn stats1d_summary_trans<'s, 'v>(
+    state: Option<Internal<StatsSummary1D<'s>>>,


It seems like the logic of all of these pg aggregate functions would be streamlined if the internal representation was a StatsSummary1DInternal. This would require a separate trans function for the rollup, but still seems like it might be cleaner.

crates/counter-agg/src/lib.rs

davidkohn88 · 2021-06-17T18:25:18Z

bors r+

bors · 2021-06-17T18:30:55Z

Build succeeded:

davidkohn88 commented May 25, 2021

View reviewed changes

davidkohn88 added 3 commits June 16, 2021 12:22

Rename regression -> stats-agg move out of counter_agg

8ddc581

adding 1d stuff

1dc96b2

after pairing

f53801b

davidkohn88 force-pushed the stats-agg branch from eb79b1f to 5fc5e8a Compare June 16, 2021 16:23

davidkohn88 marked this pull request as ready for review June 16, 2021 16:44

davidkohn88 requested a review from JLockerman June 16, 2021 16:44