@mjbvz
I hope this is the kind of change you expected. Instead of low and normal priority mark, I've used normal and high. Also, I believe that executeWithoutWaitingForResponse is of higher priority request from the user and changed accordingly.
Kindly review and let me know if I should modify things.

Thanks for the review.

specifically marking a few requests as low priority

Does that mean that we will have to insert all other request in the queue in the beginning of the array?

mark the code lens reference's request as low priority

By marking, do you mean we'll have to get the request type within the push function and based on that we'll have to insert into the queue?

Based on your review remarks, I believe that I'll have to remove the priority property that I introduced. Correct me if I'm wrong.

1. The simplest way to implement this would be to always enqueue normal priority requests in front of low priority requests. When pushing a normal priority request onto the queue, check to see if the previously queued request is a low priority one. If it is, then insert the new request in front of that one instead of at the very end of the queue.

2. I think it would be best add a new lowPriority flag to the execute function.

3. Yes, instead of a generic priority property just have a lowPriority boolean property.

Let me know if that makes sense

I hope I got your points.

• Additionally, I've considered multiple lowPriority requests at the end of the queue and inserting requests before them.
• Only the execute function has the flag lowPriority set to true.

Let me know if anything should be updated further.

@mjbvz

hook this up to mark the references lens requests

I couldn't get to the point where the references codelens request is being made to add the flag. Could you please guide me a little on this one. (although I tried to trace)

Just saw your comment. Yeah I just realized that we don't directly call execute in this path anymore and instead use a command that invokes this indirectly: https://github.com/Microsoft/vscode/blob/4b8f928808f3fbf4745702d3b558e8ccbe2a726a/extensions/typescript-language-features/src/features/referencesCodeLens.ts#L21

This was a fairly recent change: dc47417

I'll merge this PR in as is and see what we want to do about the references requests