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) privateremoved? -
@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 needgeo_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.