Replace IList<> with IEnumerable<>

[acordmike]

I did not see any specific IList<> functionality (at least for the bulk insert). It would be helpful to change this the bulk input parameters to IEnumerable<> to prevent having to wrap collections into List objects to use the methods.

[borisdj]

Implemented with 2.4.2

[borisdj]

Reverted to IList with 2.4.3 because of UpdateEntitiesIdentity(SetOutputIdentity).
Related to #76
Reference of List is required:

EFCore.BulkExtensions/EFCore.BulkExtensions/TableInfo.cs

Lines 458 to 462 in c0b9aa5

	else // Clears entityList and then refills it with loaded entites from Db
	{
	entities.Clear();
	((List<T>)entities).AddRange(entitiesWithOutputIdentity);
	}

[Mithras]

Having IList for bulk operations doesn't make much sense to me. If I need to insert millions of rows, am I supposed to load them all in memory? I guess I can still split rows in reasonable chunks and call bulkInserts multiple times but it's kinda weird that I have to do that.

[cda963]

Reverted to IList with 2.4.3 because of UpdateEntitiesIdentity(SetOutputIdentity).
Related to #76
Reference of List is required:

EFCore.BulkExtensions/EFCore.BulkExtensions/TableInfo.cs

Lines 458 to 462 in c0b9aa5

	else // Clears entityList and then refills it with loaded entites from Db
	{
	entities.Clear();
	((List<T>)entities).AddRange(entitiesWithOutputIdentity);
	}

Can't you just assign

entities = entitiesWithOutputIdentity

and keep using IEnumerable instead of IList?

In my scenario I'm reading sequentially from a .csv file and I really want to avoid using up too much memory by loading up all the rows into memory...

[Numpsy]

entities = entitiesWithOutputIdentity

It needs to update the callers view of the elements rather than just the local reference?

[cda963]

entities = entitiesWithOutputIdentity

It needs to update the callers view of the elements rather than just the local reference?

Closing this issue still doesn't solve the problem when the data is read sequentially from a file for example. IList loads everything into memory... so for bulk operations, as others have mentioned it's not very helpful.

[borisdj]

Bulk methods are now receiving IEnumerable.