Updated comment on error condition.

JordanMarr · May 22, 2024 · 372f102 · 372f102
1 parent 30631d8
commit 372f102
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/SqlHydra.Query/SelectBuilders.fs b/src/SqlHydra.Query/SelectBuilders.fs
@@ -129,7 +129,9 @@ type SelectBuilder<'Selected, 'Mapped> () =
                     //    |> Array.map (fun p -> $"%s{tableAlias}.%s{p.Name}")
                     //q.Select(props)
 
-                    // Bug fix: temporarily revert to * until option types are property implemented.
+                    // Bug fix: temporarily revert to * until option types are properly implemented.
+                    // `tableType` was not properly unwrapping option types, causing a runtime error.
+                    // For example, left joining a table creates an option type, which should be unwrapped.
                     q.Select($"%s{tableAlias}.*")
 
                 | LinqExpressionVisitors.SelectedColumn (tableAlias, column) ->