Beware, changing an API from non-streaming to streaming changes how the client interacts with the call. Any clients (e.g. Open-ILS/web/js/dojo/openils/CopyLocation.js) that call the API will need to be taught to expect a stream instead of a single array of things.
Also, while we're addressing the issue of fetching too-large data blobs, I suggest we add {substream => 1} to the underlying cstore editor call so it also fetches the big list of stuff via streaming call from cstore.
Removing pullrequest until at least the first issue is resolved.
Beware, changing an API from non-streaming to streaming changes how the client interacts with the call. Any clients (e.g. Open-ILS/ web/js/ dojo/openils/ CopyLocation. js) that call the API will need to be taught to expect a stream instead of a single array of things.
Also, while we're addressing the issue of fetching too-large data blobs, I suggest we add {substream => 1} to the underlying cstore editor call so it also fetches the big list of stuff via streaming call from cstore.
Removing pullrequest until at least the first issue is resolved.