This is the only sensible thing to do for bug 1218537 (where we want to reference this table), but doing it on it's own makes sense too, as it lowers the complexity of the function. I took the opportunity to do a bit more error checking/logging in the artifact processing code.
Attachment #8749280 - Flags: review?(cdawson)
Comment on attachment 8749280 [details] [review] [treeherder] wlach:1270561 > mozilla:master lgtm! :)
Attachment #8749280 - Flags: review?(cdawson) → review+
Pull request has landed in master: https://github.com/mozilla/treeherder/commit/c48853efedefc78bb8eb4e022dec0c24e70ffe2a
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.