Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

introduce data aggregation (#194) #234

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from
Draft

introduce data aggregation (#194) #234

wants to merge 3 commits into from

Commits on Dec 15, 2022

  1. add hits column to statify table 

    This column will be used for aggregation and is filled with 1 initially.
The number of hits for a specific date/target/referrer is thus no longer
the number of entries, but the sum of all hits.
    @stklcode
    stklcode committed Dec 15, 2022
    Configuration menu
    Copy the full SHA
    2909baa View commit details
    Browse the repository at this point in the history

  2. introduce data aggregation routine 

    We now aggregate statistics data in the cleanup routine preserving
only distinct entries for each date/referrer/target combination with
the sum of all hits.
    @stklcode
    stklcode committed Dec 15, 2022
    Configuration menu
    Copy the full SHA
    79ce9a9 View commit details
    Browse the repository at this point in the history

  3. add "statify__skip_aggregation" hook to disable aggregation 

    If Statify is extended by custom columns the aggregation routine will
fail. To support such scenarios, we introduce a boolean hook, so the
previous behavior can be preserved without breaking compatibility.
    @stklcode
    stklcode committed Dec 15, 2022
    Configuration menu
    Copy the full SHA
    e789310 View commit details
    Browse the repository at this point in the history