Only show throttling warning when throttling happened. (#1976)

Azure · Sep 23, 2024 · 7128133 · 7128133
1 parent 053dc9d
commit 7128133
Showing 1 changed file with 15 additions and 9 deletions.
diff --git a/src/Explorer/Tabs/DocumentsTabV2/DocumentsTabV2.tsx b/src/Explorer/Tabs/DocumentsTabV2/DocumentsTabV2.tsx
@@ -645,6 +645,7 @@ export const DocumentsTabComponent: React.FunctionComponent<IDocumentsTabCompone
     throttledIds: DocumentId[];
     failedIds: DocumentId[];
     beforeExecuteMs: number; // Delay before executing delete. Used for retrying throttling after a specified delay
+    hasBeenThrottled: boolean; // Keep track if the operation has been throttled at least once
   }>(undefined);
   const [bulkDeleteOperation, setBulkDeleteOperation] = useState<{
     onCompleted: (documentIds: DocumentId[]) => void;
@@ -754,6 +755,7 @@ export const DocumentsTabComponent: React.FunctionComponent<IDocumentsTabCompone
             throttledIds: newThrottled,
             failedIds: prev.failedIds.concat(newFailed),
             beforeExecuteMs: retryAfterMilliseconds,
+            hasBeenThrottled: prev.hasBeenThrottled || newThrottled.length > 0,
           }));
         })
         .catch((error) => {
@@ -764,6 +766,7 @@ export const DocumentsTabComponent: React.FunctionComponent<IDocumentsTabCompone
             successfulIds: prev.successfulIds,
             failedIds: prev.failedIds.concat(prev.pendingIds),
             beforeExecuteMs: undefined,
+            hasBeenThrottled: prev.hasBeenThrottled,
           }));
           bulkDeleteOperation.onFailed(error);
         });
@@ -1142,6 +1145,7 @@ export const DocumentsTabComponent: React.FunctionComponent<IDocumentsTabCompone
         successfulIds: [],
         failedIds: [],
         beforeExecuteMs: 0,
+        hasBeenThrottled: false,
       });
       setIsBulkDeleteDialogOpen(true);
       setBulkDeleteMode("inProgress");
@@ -2312,15 +2316,17 @@ export const DocumentsTabComponent: React.FunctionComponent<IDocumentsTabCompone
                 </MessageBarBody>
               </MessageBar>
             )}
-            <MessageBar intent="warning">
-              <MessageBarBody>
-                <MessageBarTitle>Warning</MessageBarTitle>
-                {get429WarningMessageNoSql()}{" "}
-                <Link href={NO_SQL_THROTTLING_DOC_URL} target="_blank">
-                  Learn More
-                </Link>
-              </MessageBarBody>
-            </MessageBar>
+            {bulkDeleteProcess.hasBeenThrottled && (
+              <MessageBar intent="warning">
+                <MessageBarBody>
+                  <MessageBarTitle>Warning</MessageBarTitle>
+                  {get429WarningMessageNoSql()}{" "}
+                  <Link href={NO_SQL_THROTTLING_DOC_URL} target="_blank">
+                    Learn More
+                  </Link>
+                </MessageBarBody>
+              </MessageBar>
+            )}
           </div>
         </ProgressModalDialog>
       )}