work on review comments

microsoft · Jun 6, 2024 · 810cb16 · 810cb16
1 parent 6f8bf1c
commit 810cb16
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 6 deletions.
diff --git a/src/Microsoft.Health.Fhir.CosmosDb.Core/Features/Storage/ICollectionSetup.cs b/src/Microsoft.Health.Fhir.CosmosDb.Core/Features/Storage/ICollectionSetup.cs
@@ -3,7 +3,6 @@
 // Licensed under the MIT License (MIT). See LICENSE in the repo root for license information.
 // -------------------------------------------------------------------------------------------------
 
-using System;
 using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;

diff --git a/...ft.Health.Fhir.CosmosDb.Core/Features/Storage/StoreProcedures/IStoredProcedureMetadata.cs b/...ft.Health.Fhir.CosmosDb.Core/Features/Storage/StoreProcedures/IStoredProcedureMetadata.cs
@@ -2,9 +2,7 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License (MIT). See LICENSE in the repo root for license information.
 // -------------------------------------------------------------------------------------------------
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
+
 using Microsoft.Azure.Cosmos.Scripts;
 
 namespace Microsoft.Health.Fhir.CosmosDb.Core.Features.Storage.StoredProcedures

diff --git a/...sDb.Initialization/Features/Storage/StoredProcedures/DataPlaneStoredProcedureInstaller.cs b/...sDb.Initialization/Features/Storage/StoredProcedures/DataPlaneStoredProcedureInstaller.cs
@@ -17,15 +17,13 @@ public sealed class DataPlaneStoredProcedureInstaller : IStoredProcedureInstalle
     {
         private readonly IEnumerable<IStoredProcedureMetadata> _storeProceduresMetadata;
 
-        // TODO: refactor constructor to have dependency on container
         public DataPlaneStoredProcedureInstaller(IEnumerable<IStoredProcedureMetadata> storedProcedures)
         {
             EnsureArg.IsNotNull(storedProcedures, nameof(storedProcedures));
 
             _storeProceduresMetadata = storedProcedures;
         }
 
-        // TODO: refactor method to have dependency on IReadOnlyList<Istoredproceduremetada>
         public async Task ExecuteAsync(Container container, CancellationToken cancellationToken)
         {
             foreach (IStoredProcedureMetadata storedProc in _storeProceduresMetadata)