improve race condition comment

Azure · Jun 5, 2020 · fc1806f · fc1806f
1 parent 8e80b6e
commit fc1806f
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/sdk/formrecognizer/Azure.AI.FormRecognizer/src/CopyModelOperation.cs b/sdk/formrecognizer/Azure.AI.FormRecognizer/src/CopyModelOperation.cs
@@ -166,7 +166,7 @@ private async ValueTask<Response> UpdateStatusAsync(bool async, CancellationToke
 
                 if (update.Value.Status == OperationStatus.Succeeded)
                 {
-                    // we need to first assign a vaue and then mark the operation as completed to avoid race conditions
+                    // We need to first assign a value and then mark the operation as completed to avoid a race condition with the getter in Value
                     _value = ConvertValue(update.Value, _targetModelId, CustomFormModelStatus.Ready);
                     _hasCompleted = true;
                 }

diff --git a/sdk/formrecognizer/Azure.AI.FormRecognizer/src/FormRecognizerClient.cs b/sdk/formrecognizer/Azure.AI.FormRecognizer/src/FormRecognizerClient.cs
@@ -133,7 +133,7 @@ public virtual RecognizeContentOperation StartRecognizeContent(Stream form, Reco
             FormContentType contentType = recognizeOptions.ContentType ?? DetectContentType(form, nameof(form));
 
             ResponseWithHeaders<ServiceAnalyzeLayoutAsyncHeaders> response =  ServiceClient.AnalyzeLayoutAsync(contentType, form, cancellationToken);
-            return new RecognizeContentOperation(ServiceClient, Diagnostics,response.Headers.OperationLocation);
+            return new RecognizeContentOperation(ServiceClient, Diagnostics, response.Headers.OperationLocation);
         }
 
         /// <summary>

diff --git a/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeCustomFormsOperation.cs b/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeCustomFormsOperation.cs
@@ -155,7 +155,7 @@ private async ValueTask<Response> UpdateStatusAsync(bool async, CancellationToke
 
                 if (update.Value.Status == OperationStatus.Succeeded)
                 {
-                    // we need to first assign a vaue and then mark the operation as completed to avoid race conditions
+                    // We need to first assign a value and then mark the operation as completed to avoid a race condition with the getter in Value
                     _value = ConvertToRecognizedForms(update.Value.AnalyzeResult);
                     _hasCompleted = true;
                 }

diff --git a/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeReceiptsOperation.cs b/sdk/formrecognizer/Azure.AI.FormRecognizer/src/RecognizeReceiptsOperation.cs
@@ -123,7 +123,7 @@ private async ValueTask<Response> UpdateStatusAsync(bool async, CancellationToke
 
                 if (update.Value.Status == OperationStatus.Succeeded)
                 {
-                    // we need to first assign a vaue and then mark the operation as completed to avoid race conditions
+                    // We need to first assign a value and then mark the operation as completed to avoid a race condition with the getter in Value
                     _value = ConvertToRecognizedReceipts(update.Value.AnalyzeResult);
                     _hasCompleted = true;
                 }

diff --git a/sdk/formrecognizer/Azure.AI.FormRecognizer/src/TrainingOperation.cs b/sdk/formrecognizer/Azure.AI.FormRecognizer/src/TrainingOperation.cs
@@ -117,7 +117,7 @@ private async ValueTask<Response> UpdateStatusAsync(bool async, CancellationToke
 
                 if (update.Value.ModelInfo.Status == CustomFormModelStatus.Ready)
                 {
-                    // we need to first assign a vaue and then mark the operation as completed to avoid race conditions
+                    // We need to first assign a value and then mark the operation as completed to avoid a race condition with the getter in Value
                     _value = new CustomFormModel(update.Value);
                     _hasCompleted = true;
                 }