Add GetReportByRequestId.

[riemanli]

No description provided.

[wfa-reviewable]

This change is 

[riemanli]

Current dependencies on/for this PR:

• main
  • PR Detail the error types in createMetric and batchCreateMetrics. #1005
    • PR Add GetReportByRequestId. #1007 👈
    • PR Add createReport. #1004
      • PR Add getReport. #1029

This comment was auto-generated by Graphite.

[tristanvuong2021]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @jcorilla and @riemanli)

a discussion (no related file):
Is this necessary? CreateReport can return the existing report if one is found with the same request id.

---

[jcorilla]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @riemanli)

[riemanli]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @tristanvuong2021)

a discussion (no related file):

Previously, tristanvuong2021 (Tristan Vuong) wrote…

Is this necessary? CreateReport can return the existing report if one is found with the same request id.

You're right. I thought I could save some computation before calling internal CreateReport by using GetReportByRequestId, but it turned out there is not much to save. I will close this PR.

---