Don't call Azure queue GetProperties API unnecessarily#2613
Don't call Azure queue GetProperties API unnecessarily#2613JorTurFer merged 5 commits intokedacore:mainfrom
Conversation
This PR reorders the logic of getting an Azure queue size in a way that skips the call to get the queue properties (for getting the ApproximateMessagesCount) if the queue has less than 32 messages. Signed-off-by: Ram Cohen <ram.cohen@gmail.com>
|
/run-e2e azure-queue* |
|
/run-e2eazure-queue.test* |
JorTurFer
left a comment
There was a problem hiding this comment.
LGTM!
We will keep this PR open till we have released v2.6.1 because we'd like to release in it only bugfixes.
When we have release it, I'll ping you again for updating the changelog.
Thanks for your contribution ❤️
JorTurFer
left a comment
There was a problem hiding this comment.
LGTM!
We will keep this PR open till we have released v2.6.1 because we'd like to release in it only bugfixes.
When we have release it, I'll ping you again for updating the changelog.
Thanks for your contribution ❤️
|
hey @RamCohen |
Signed-off-by: Ram Cohen <ram.cohen@gmail.com>
Signed-off-by: Ram Cohen <ram.cohen@gmail.com>
|
There is a conflict in the changelog. |
Signed-off-by: Ram Cohen <ram.cohen@gmail.com>
Signed-off-by: Ram Cohen <ram.cohen@gmail.com>
This PR reorders the logic of getting an Azure queue size in a way
that skips the call to get the queue properties (for getting the
ApproximateMessagesCount) if the queue has less than 32 messages.
Signed-off-by: Ram Cohen ram.cohen@gmail.com