experimenter icon indicating copy to clipboard operation
experimenter copied to clipboard

Show per-metric analysis timestamps on Results page

Open mikewilli opened this issue 2 years ago • 1 comments

Show the timestamp of analysis on a per-metric basis, which we should be able to grab from BQ along with the error metadata.

This is a nice-to-have in addition to #7474 (which will show the overall start time of a jetstream run), but should be pretty simple to do because we will already be grabbing the error metadata.

┆Issue is synchronized with this Jira Task

mikewilli avatar Jul 19 '22 19:07 mikewilli

cc @danielkberry

mikewilli avatar Jul 19 '22 19:07 mikewilli

Bump

jaredlockhart avatar Feb 09 '23 15:02 jaredlockhart

I'm not sure if this will provide value beyond what we currently have with the Analysis Start Time, and I worry that it could cause additional confusion. If you think it's valuable let's discuss. Did this come up again recently?

mikewilli avatar Feb 09 '23 15:02 mikewilli

As I understand it, this issue happens because Jetstream output can be in an inconsistent state if certain configs are pushed at certain times. If Jetstream implements this proposal to drop all tables on a rerun, then this shouldn't be necessary. (Presumably as part of that work we would make sure the GCP output is dropped and rewritten as well)

cc @scholtzan

danielkberry avatar Feb 09 '23 16:02 danielkberry

Makes sense, I thought about that issue and think it would be a better way to resolve this concern. And the output sent for ingest by Experimenter is aggregated so there should not be a drop-and-rewrite needed there.

mikewilli avatar Feb 09 '23 18:02 mikewilli

The proposal @danielkberry mentioned has been implemented in Jetstream (see here), so I would propose we cancel this ticket unless we think it is still valuable.

mikewilli avatar Apr 14 '23 18:04 mikewilli

Yeah, this can be closed

scholtzan avatar Apr 20 '23 07:04 scholtzan