Looks reasonable. Not totally fond of the hard-wired constant there.
I'm not going to hold up this MR because of it. But should we make that a config item? If there is more than one place where that number is used, we probably should... If so, could you open a separate bug please?
Looks reasonable. Not totally fond of the hard-wired constant there.
I'm not going to hold up this MR because of it. But should we make that a config item? If there is more than one place where that number is used, we probably should... If so, could you open a separate bug please?