Deprecate Redis, RedisHLL and Snowplow Metrics instrumentation
Problem
With the Gitlab Internal Events Tracking we are aiming to create a replacement for the disparate Redis, RedisHLL and Snowplow metrics instrumentation we offer. Under the hood the same technology will be used but the API for developers will be unified into one place that selects the right technology based on the use-case.
If we don't point our users to the new API and deprecate the old, separate methods, they will still continue to use the old methods, either out of habit or simply because they don't know there's an alternative.
Desired Outcome
- Separate Snowplow, Redis, RedisHLL methods are clearly marked as deprecated in the code
- Remove the
generate gitlab:snowplow_event_definition
rake task - Dangerbot warns people if they instrument new metrics with
data_source
beingredis
orredis_hll
.
Proposed solution
Use Dangerbot and mark the code as deprecated. Old documentation is getting removed as part of gitlab-org/analytics-section/analytics-instrumentation/internal#670 (closed)
How to verify
Create an example MR for the different methods to instrument and make sure to get warned accordingly