Proposed Fix for Issue #41

[stewart-r]

I believe this fixes #41 .

Instead of simply using a batch size of 100 we now check the maximum possible size of the entity type we are given and calculate the batch size from there by dividing the max batch size by the max size of an entity then using the min of that result and 100.

in order to calculate the max size of an entity I used the table here

I did have a concern that as I am measuring only the properties on a DynamicTableEntity that later elements in the array may have a different set of properties but, on reflection, I don't think this is possible. I think it's impossible because, as it is being called from inside the Insert function, the DynamicTableEntity will have just been mapped from the generated table entity?

[isaacabraham]

Thanks for this - I was dreading doing it :-) I'm a bit busy over the next couple of days so apologies - but I will get around to reviewing this (and then no doubt accepting the PR).

[stewart-r]

Thanks Isaac.

No need to rush and certainly no need apologise for being busy! :-)