Support for dynamic SQL with variable parameters in AOT mode #157

Activity

1. DL444 commented on Jul 11, 2025

   

   DL444

   on Jul 11, 2025

   Last edited by DL444

   More actions

   I don't think dynamic parameterization is inherently incompatible with Native AOT, just that someone needs to get it done.

   For example, I imagine that we can take the generated command factory code quoted by the docs:

   public override void AddParameters(in global::Dapper.UnifiedCommand cmd, object? args)
   {
       var typed = Cast(args, static () => new { productId = default(int) }); // expected shape
       var ps = cmd.Parameters;
       global::System.Data.Common.DbParameter p;
       p = cmd.CreateParameter();
       p.ParameterName = "productId";
       p.DbType = global::System.Data.DbType.Int32;
       p.Direction = global::System.Data.ParameterDirection.Input;
       p.Value = AsValue(typed.productId);
       ps.Add(p);
   }

   and adapt it into this so dictionary-typed parameters could be supported:

   public override void AddParameters(in global::Dapper.UnifiedCommand cmd, object? args)
   {
       var typed = (IDictionary<string, object>)args;
       var ps = cmd.Parameters;
       global::System.Data.Common.DbParameter p;
       p = cmd.CreateParameter();
       p.ParameterName = "productId";
       p.DbType = global::System.Data.DbType.Int32;
       p.Direction = global::System.Data.ParameterDirection.Input;
       p.Value = typed["productId"];
       ps.Add(p);
   }

   Support for DynamicParameter can be added in a similar way. It won't and can't cover every parameter type possible, but should at least offer some ways to unblock users. I might see if it can actually be done later when I have time.

   ---

   EDIT: I just noticed that the issue asked for not just dynamic parameters but dynamic SQL as well. This would be difficult since the query cannot be statically analyzed. If there are only a handful of combinations of queries possible, maybe splitting them each into a static SQL command would be the solution. Otherwise I don't really have a good idea.