I'm not sure about this assert. In general, I'm not sure that we compile asserts in on dev builds and out on production builds - if we want to use asserts, we should do that. In this case specifically, I think it'll be easier to debug if we carry the processing on and have warning/error level logs with the timer's details, the bucket we expected it in, and the bucket it was actually in.