doStreamGetQuery returns only 100 records when limit>100

Hello , is there anything that is blocking the release of RMB with this fix?

this fix will need to be a part of Q3 2020 release.

My sincere apologies, it's been a while since I've read Java. Glad that it's not as pervasive a problem as that.

doStreamGetQuery is NOT used by ResultInfo.setTotalRecords.

This is the code taken from raml-module-builder/domain-models-api-interfaces/target/generated-sources/raml-jaxrs/org/folio/rest/jaxrs/model/ResultInfo.java (use mvn compile to generate the code):

doStreamGetQuery calls setTotalRecords, setTotalRecords does NOT call doStreamGetQuery.

PostgresClient's get and doGet methods use processResults that calls setTotalRecords: https://github.com/folio-org/raml-module-builder/blob/v31.1.1/domain-models-runtime/src/main/java/org/folio/rest/persist/PostgresClient.java#L2600 They do not call doStreamGetQuery.

doStreamGetQuery is used in setTotalRecords in RMB (https://github.com/search?q=org%3Afolio-org+doStreamGetQuery&type=code), which is used in over 125 places in the code on GitHub (https://github.com/search?q=org%3Afolio-org+setTotalRecords&type=code). One of those places is mod-calendar, where it looks like it's possible that if there were over 100 events or exceptions, they would not all be returned and thus the due dates of materials COULD be off by a number of days. This is getting pretty out there in terms of "what if", but it shows that this bug could have implications on the data in a production system.