Use `state_machine` in `MergeTrain`
Currently, MergeTrain
object is not using state_machine
. We should use it when the event hook gets complicated.
The following discussion from !19434 (merged) should be addressed:
-
@cablett started a discussion: (+1 comment) Can we rename this method (or make a method alias) to something like
mark_as_stale!
ormark_for_refresh
or something similar? From an abstraction perspective,stale!
is also callingrefresh_async
so maybe the alias could capture that so it's clear what it's doing.mark_stale_and_force_refresh
orforce_refresh
might be good, WDYT?