Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

New Functionalities Added #165

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
2 participants

Hey Rob,

I have added a few methods to Massive that I think bring minor enhancements

  1. Ability to convert a collection of dynamics to a collection of concrete type
  2. Convert a collection of dynamics to DataTable based on the DynamicModel
  3. Bulk insert the dynamic collection to SQL Server. This should probably be encapsulated in its own file

Also, I have added an example of every functionality that I have added.

Thanks,

Ashar

It works fine for us but we are loading no more than 200 rows of data. Didn't think to optimize the code further since it met our needs.

Ran a few tests today using the AdventureWorks database on mid 2007 laptop. Below tests do not include the load time, just the cast time

504 Products got materialized and casted in 16 ms
19972 Persons got materialized and casted in 110 ms

Contributor

robconery commented Jan 8, 2013

I really appreciate the love here but Massive doesn't do type-casting for a very specific reason: it will quickly become a support nightmare. I learned this from SubSonic - that's why Massive is all Dynamic.

That said - If you'd like to contribute a separate "Helper" file or something, that would be really nice of you - but I don't believe this belongs in core.

@robconery robconery closed this Jan 8, 2013

I have broken out the functionality into a separate file. Please see if this may be acceptable and of use to the masses. Thanks!

Date: Tue, 8 Jan 2013 14:45:08 -0800
From: notifications@github.com
To: massive@noreply.github.com
CC: asharz@outlook.com
Subject: Re: [massive] New Functionalities Added (#165)

I really appreciate the love here but Massive doesn't do type-casting for a very specific reason: it will quickly become a support nightmare. I learned this from SubSonic - that's why Massive is all Dynamic.

That said - If you'd like to contribute a separate "Helper" file or something, that would be really nice of you - but I don't believe this belongs in core.

          —

          Reply to this email directly or view it on GitHub.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment