- Issue created by @skyriter
- Status changed to Postponed
2 months ago 11:38pm 30 June 2024 - πΊπΈUnited States swirt Florida
Interesting find. I think we should table this one in favor of π Remove examples and references to getBatchSize() Active
The batch setting right now is really an illusion and there is no benefit in using it in 99% of the uses cases.It is too hard to explain both how to do the remaining 1% and show an example of how to actually try it do anything better than a batch size of 1.
- Status changed to Fixed
2 months ago 1:12am 1 July 2024 - πΊπΈUnited States swirt Florida
Thank you for finding this @skyriter and reporting it.
I actually resolved this as part of π Remove examples and references to getBatchSize() Active
It turned out we were passing a formatted number string so anything under 1000 got interpreted as an int, but then at 1,000 it got a comma added and became a string.
I altered the code to pass in the raw value and not the formatted value. - Status changed to Fixed
2 months ago 4:16am 1 July 2024