#3402 Correct getAverageDuration values for most GC builds
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue1128  into  master

Download 3402.patch

Fixes: https://pagure.io/koji/issue/1128

I was a bit afraid of query speed but it is almost the same query plan - just extended filter on build_by_pkg_id index scan. Nevertheless, execution time grows ~10x. Random package (10ms vs 94ms) It seems acceptable as it is not called too often.

rebased onto 47d3f89ae39aa644c51add8a14624dbfa9167c80

pretty please pagure-ci rebuild

It's not an idea solution, but I guess it's the best we can do until we get a better solution for associating tasks with CG builds.

It seems acceptable as it is not called too often.

Specifically, it is called:

  • at the start of each buildArch task. A few milliseconds here makes no real difference.
  • for each load of a taskinfo and buildinfo pages in the web ui. This is a much shorter timescale than buildArch, but still not an issue to add a few milliseconds.
  • Ken's chatbot plugin, as described in #1128. Similarly ok.

It's possible that another outside system could be using this call, but I would expect a similar usage pattern if so.

:thumbsup:

Metadata Update from @tkopecek:
- Pull-request tagged with: no_qe

Commit b5ff9d8b fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata