Support site-specific aggregation

storborg / manhattan

A gentleman's cocktail, for civilized analytics and testing

Other

2 stars 3 forks source link

Support site-specific aggregation #1

Closed storborg closed 12 years ago

storborg commented 12 years ago

As of now the frontend and logs support site-specific data, but the backend aggregator does not.

Denormalized results should be available per-site, using the .site_id attribute available on each record. This could be implemented either:

Explicitly in each Rollup instance as desired
Implicitly by building additional rollups for each site
Implicitly by performing each rollup for each site
???

storborg commented 12 years ago

Done in 0dbddec6b960bfb0c22dd948b3540a685c0b675b, by including the site id in the key alongside rollups--this means that rollup aggregation is always site specific, but that it never has to be done explicitly inside the Rollup instance.