Bug 1882110 Comment 1 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

It's possible this was always happening and error hand-off was ???? due to it being GVST

FWIW I have confirmed that we collected data in this metric before migration:

```SQL
SELECT
  m.client_info.client_id,
  submission_timestamp,
  ARRAY_LENGTH(metrics.timing_distribution.geckoview_page_load_time.values) as kv_pair_counts,
  client_info.app_display_version
  -- plt.*
FROM
  org_mozilla_firefox.metrics AS m
-- cross join
--  unnest (metrics.timing_distribution.geckoview_page_load_time.values) as plt
WHERE
  DATE(submission_timestamp) >= DATE_SUB(CURRENT_DATE(), INTERVAL 60 DAY)
  and ARRAY_LENGTH(metrics.timing_distribution.geckoview_page_load_time.values) > 0
  and mozfun.norm.extract_version(client_info.app_display_version, "major") < 124
  and mozfun.norm.extract_version(client_info.app_display_version, "major") > 120

LIMIT 100
```

This error occurs when `.finish()` is called, when I look at the code it appears that `start()` should HAVE to get called for us to get to the Glean call site, so I need to investigate what might be going on. 

Thoughts:
ID collision? start twice, but only one id, try to cancel twice?

starting glean failed?

```js
  start() {
    if (this.isRunning()) {
      this.cancel();
    }
    this._timerId = this._metric.start();
  }
```

pretty straightforward, so maybe we're failing inside of `.start()` OR maybe this.isRunning() isn't always cleared correctly
It's possible this was always happening and error hand-off was ???? due to it being GVST

FWIW I have confirmed that we collected data in this metric before migration:

```SQL
SELECT
  m.client_info.client_id,
  submission_timestamp,
  ARRAY_LENGTH(metrics.timing_distribution.geckoview_page_load_time.values) as kv_pair_counts,
  client_info.app_display_version
  -- plt.*
FROM
  org_mozilla_firefox.metrics AS m
-- cross join
--  unnest (metrics.timing_distribution.geckoview_page_load_time.values) as plt
WHERE
  DATE(submission_timestamp) >= DATE_SUB(CURRENT_DATE(), INTERVAL 60 DAY)
  and ARRAY_LENGTH(metrics.timing_distribution.geckoview_page_load_time.values) > 0
  and mozfun.norm.extract_version(client_info.app_display_version, "major") < 124
  and mozfun.norm.extract_version(client_info.app_display_version, "major") > 120

LIMIT 100
```

This error occurs when `.finish()` is called, when I look at the code it appears that `start()` should HAVE to get called for us to get to the Glean call site, so I need to investigate what might be going on. 

Thoughts:
ID collision? start twice, but only one id, try to cancel twice? _seems unlikely_

starting glean failed?

```js
  start() {
    if (this.isRunning()) {
      this.cancel();
    }
    this._timerId = this._metric.start();
  }
```

pretty straightforward, so maybe we're failing inside of `.start()` OR maybe `this.isRunning()` isn't always cleared correctly

Back to Bug 1882110 Comment 1