linkedin/parseq

Split existing batches according to maxBatchSize configuration in ParSeqRestClient

Closed this issue · 1 comments

Currently maxBatchSize does not affect existing batch GET requests. It is only used to limit size of batch requests created as a result of aggregation, see:

if (_batchSize == 0 || (safeToAddWithoutOverflow(_batchSize, size) && _batchSize + size <= _maxSize)) {

This change is to make sure that all outgoing batch GET requests have size below maxBatchSize even if original batch GET request had a larger size.

We have decided to document this in parseq-restli-client README.md for this behavior, this has been committed in master.