Adding back in fix for WebJobs Extension Queue Length and updated cha…

…ngelog (Azure#45968)
chiangvincent · Sep 13, 2024 · a11418b · a11418b
1 parent 2051661
commit a11418b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/...crosoft.Azure.WebJobs.Extensions.Storage.Common/src/Shared/Queues/QueueMetricsProvider.cs b/...crosoft.Azure.WebJobs.Extensions.Storage.Common/src/Shared/Queues/QueueMetricsProvider.cs
@@ -37,8 +37,8 @@ public async Task<int> GetQueueLengthAsync()
         {
             try
             {
-                QueueProperties queueProperties = await _queue.GetPropertiesAsync().ConfigureAwait(false);
-                return queueProperties.ApproximateMessagesCount;
+                QueueTriggerMetrics queueMetrics = await GetMetricsAsync().ConfigureAwait(false);
+                return queueMetrics.QueueLength;
             }
             catch (RequestFailedException ex)
             {

diff --git a/sdk/storage/Microsoft.Azure.WebJobs.Extensions.Storage.Queues/CHANGELOG.md b/sdk/storage/Microsoft.Azure.WebJobs.Extensions.Storage.Queues/CHANGELOG.md
@@ -2,20 +2,17 @@
 
 ## 5.4.0-beta.1 (Unreleased)
 
-### Features Added
-
-### Breaking Changes
-
 ### Bugs Fixed
+- When grabbing Queue Metrics for amount of messages, will now use the QueueTriggerMetrics.QueueLength instead of the ApproximateMessagesCount for less stale metrics.
 
 ### Other Changes
+- Improvement in logging exceptions when retrieving queue metrics.
 
 ## 5.3.1 (2024-07-17)
 
 ### Bugs Fixed
 - Rely on PeekMessagesAsync when calculating message queue length
 - Fixing target base scale instance concurrency for queues
-- When grabbing Queue Metrics for amount of messages, will now use the QueueTriggerMetrics.QueueLength instead of the ApproximateMessagesCount for less stale metrics.
 
 ## 5.3.0 (2024-04-18)
 - Includes all features from 5.3.0-beta.1.