1505793 - Add triage owner in /rest/bug

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: API, enhancement)

Description

[Calixte Denizet (:calixte)]

We should have an entry for triage_owner instead of querying /rest/product to get it.

Comment 1

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request

Comment 2

[Calixte Denizet (:calixte)]

If we could have triage_owner_detail as we've for creator then it'd be nice (to avoid to make a query on user).

Comment 3

[Calixte Denizet (:calixte)]

:kohei, you guessed my needs, thx (I didn't read your patch before doing my comment).

Comment 4

[Kohei Yoshino [:kohei]]

Yes, triage_owner_detail is there :)

Comment 5

[David Lawrence [:dkl]]

Merged to master.

Comment 6

[Dylan Hardison [:dylan] (he/him)]

This cannot work like this.

We can return a new top level thing, triage_owners, that is indexed by product and component but adding this to each bug result will use fantastically more memory

Comment 7

[David Lawrence [:dkl]]

Reopening. This will need to be reimplemented differently in order to not increase memory usage for the API.

Comment 8

[David Lawrence [:dkl]]

Attachment: GitHub Pull Request to revert the initial PR

Comment 9

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request, v2

This makes `triage_owner` an extra field so you’ll have to use the `include_fields` param like this:

https://bugzilla.mozilla.org/rest/bug/1505793?include_fields=_default,_extra
https://bugzilla.mozilla.org/rest/bug/1505793?include_fields=_default,triage_owner
https://bugzilla.mozilla.org/rest/bug/1505793?include_fields=id,summary,triage_owner
(or whatever you want)

Comment 10

[David Lawrence [:dkl]]

Merged to master. again.

Comment 11

[Calixte Denizet (:calixte)]

thanks guys, it's ok for me like this.

Comment 12

[Dylan Hardison [:dylan] (he/him)]

So memory usage has exploded because people ask for _custom all the time

Comment 13

[Kohei Yoshino [:kohei]]

But _custom won’t return triage owners. Just _extra or triage_owner.

Comment 14

[Kohei Yoshino [:kohei]]

Also user objects are cached and there aren’t so many triage owners, right?

Comment 15

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Dylan mentioned using an intermediate layer like GraphQL to cache this information. That layer could do the join, or consuming applications could just fetch the list of Triage Owners and do the join on their own.

Comment 16

[Dylan Hardison [:dylan] (he/him)]

plenty of API customers use _extra and _custom, but for the moment this can ride.

Something like GraphQL would help, because it would encourage people to use paging. Right now some customers expect 4000-5000 or more bugs per request.