PR review changes

FrendsPlatform · Feb 5, 2025 · 9eea415 · 9eea415
1 parent 3f2cf96
commit 9eea415
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/Frends.MicrosoftSQL.BulkInsert/Frends.MicrosoftSQL.BulkInsert/BulkInsert.cs b/Frends.MicrosoftSQL.BulkInsert/Frends.MicrosoftSQL.BulkInsert/BulkInsert.cs
@@ -105,12 +105,9 @@ private static async Task<long> ExecuteHandler(Options options, Input input, Dat
     {
         var rowsCopied = 0L;
 
-
-        if (input.ColumnMapping == ColumnMapping.JsonPropertyOrder)
-        {
-            // ColumnMapping.JsonPropertyOrder is default and is already being handled by SqlBulkCopy.
-        }
-        else if (input.ColumnMapping == ColumnMapping.JsonPropertyNames)
+        // JsonPropertyOrder is handled implicitly (default behavior) by not adding any column mappings,
+        // which means the columns will be mapped based on their order in the input JSON.
+        if (input.ColumnMapping == ColumnMapping.JsonPropertyNames)
         {
             foreach (var column in dataSet.Tables[0].Columns)
                 sqlBulkCopy.ColumnMappings.Add(new SqlBulkCopyColumnMapping(column.ToString(), column.ToString()));