Gerke, Both of these have been pulled in and I released 1.2.4 on Nuget that includes them (with the exception of defaulting SqlClient when nothing is provided.)
Nathan Palmer On Fri, Dec 13, 2013 at 3:20 AM, Gerke Geurts <ggeu...@gmail.com> wrote: > Hello, > > I have submitted pull requests with unit tests and patches for a couple of > issues I encountered while developing a Rhino.Etl based solution: > > - The new SortMergeJoinOperation class can loose the final rows in > outer joins if multiple rows at the end of either the left or right side > don't match up. Pull request > https://github.com/hibernating-rhinos/rhino-etl/pull/14 fixes this > problem. > - The use of ProviderName in > System.Configuration.ConnectionStringSettings instances does not match > up with regular ADO.NET use. Rhino.Etl expects the assembly qualified > name of a connection class > (System.Data.Common.IDbConnectionimplementation) whereas the intended use > seems to be to specify a data > provider name that can be mapped to a > System.Data.Common.DbProviderFactory using the > System.Data.Common.DbProviderFactories class (see > http://msdn.microsoft.com/en-us/library/ms971499.aspx). A connection > instance can than be obtained by calling the > DBProviderFactory.CreateConnection() method. Pull request > https://github.com/hibernating-rhinos/rhino-etl/pull/15 modifies the > Rhino.Etl.Infrastructure.Use class to support the data provider model > while maintaining backwards compatibility. > > I hope these pull requests can be integrated in the current code base. > > Kind Regards, > Gerke. > > -- > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to rhino-tools-dev+unsubscr...@googlegroups.com. > To post to this group, send email to rhino-tools-dev@googlegroups.com. > Visit this group at http://groups.google.com/group/rhino-tools-dev. > For more options, visit https://groups.google.com/groups/opt_out. > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to rhino-tools-dev+unsubscr...@googlegroups.com. To post to this group, send email to rhino-tools-dev@googlegroups.com. Visit this group at http://groups.google.com/group/rhino-tools-dev. For more options, visit https://groups.google.com/groups/opt_out.
