Follow-up from "Track the Geo event log gaps in redis and handle them later"

The following discussions from !6640 (merged) should be addressed:

  • @ash.mckenzie started a discussion: (+3 comments)

    private removed?

  • @ash.mckenzie started a discussion: (+2 comments)

    An example here of some values might assist?

  • @stanhu commented on a discussion: (+2 comments)

    I think we still need Gitlab::Geo::LogHelpers? I think @ash.mckenzie was saying we don't need geo_logger. I'll revert your last commit and remove that method.

  • @stanhu started a discussion:

    This could be an N+1 query. We might need to do this in batches.

Assignee Loading
Time tracking Loading