-
Notifications
You must be signed in to change notification settings - Fork 248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: introduce new API for batch task requests #6915
Conversation
I'm submitting this to get some feedback to make sure I'm headed in the right direction. I have a couple of questions inside (in XXX) about auth and validation, I'd be glad to get some pointers there. :) |
cd4f45b
to
8cc3ed2
Compare
8cc3ed2
to
72c8b5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've updated the commit following your advice. I think this is starting to look good. Should I get it out of draft?
72c8b5f
to
77f4228
Compare
assert.equal(rows.length, expectedIndexes.length); | ||
assert.deepStrictEqual(rows, expectedTasks); | ||
|
||
// Order of the input doesn't matter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! This looks great, thanks for the contirbution
This is done according to RFC 189¹. In few words, the motivation is to drastically reduce the number of index API calls done by decision tasks, since they sometimes need the task ids for hundreds of tasks. [1] taskcluster/taskcluster-rfcs#189
77f4228
to
b59457e
Compare
This is done according to RFC 189¹. In few words, the motivation is to drastically reduce the number of index API calls done by decision tasks, since they sometimes need the task ids for hundreds of tasks.
[1] taskcluster/taskcluster-rfcs#189