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.

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

[hotfix][ENG-1963][ENG-2010] Keen logging updates #9441

Merged

Conversation

felliott
Copy link
Member

Purpose

  • Stop logging unused events to keen
  • Log public pageview into partitioned collections for smaller/faster scans

Changes

  • Stop logging front-end-events to Keen. These should only be sent to GA.
  • Start sending pageviews for public projects to collections partitioned by the first letter of the guid. These collections will be created automatically by Keen. We will switch over to them after collecting a month of data after which scans should be much faster.

QA Notes

  • No data migration required.
  • Fix will be manually dev-tested.

Documentation

No documentation needs to be updated.

Side Effects

Unlikely. Changes are highly restricted to keen tracking code.

Ticket

https://openscience.atlassian.net/browse/ENG-1963
https://openscience.atlassian.net/browse/ENG-2010

 * We don't use this collection anymore, so stop wasting time logging
   it.

   [ENG-1963]
 * Log the public pageview events into collection buckets partitioned
   by the first letter of the project guid.  After one month, we will
   switch the analytics page to pull from the partitioned buckets.
   Since each bucket will have approximately 1/31 (there are 31
   possible letters or numbers that can appear in a guid) of the
   events in the common collection, it should make the analytics page
   more responsive.

   [ENG-2010]
@felliott felliott changed the title [WIP][ENG-1963][ENG-2010] Keen logging updates [hotfix][ENG-1963][ENG-2010] Keen logging updates Jul 22, 2020
Copy link
Member

@mfraezz mfraezz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mfraezz mfraezz merged commit aeba901 into CenterForOpenScience:master Jul 22, 2020
@felliott felliott deleted the hotfix/keen-logging-changes branch August 10, 2022 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants