Bug #100415 Contribution: Add maxResultBuffer property.
Submitted: 3 Aug 2020 14:37 Modified: 3 Aug 2020 19:41
Reporter: OCA Admin (OCA) Email Updates:
Status: Verified Impact on me:
None 
Category:Connector / J Severity:S3 (Non-critical)
Version: OS:Any
Assigned to: CPU Architecture:Any

[3 Aug 2020 14:37] OCA Admin
Description:
This bug tracks a contribution by Marcin Szymborski (Github user: marszymb) as described in http://github.com/mysql/mysql-connector-j/pull/50

How to repeat:
See description

Suggested fix:
See contribution code attached
[3 Aug 2020 14:37] OCA Admin
Contribution submitted via Github - Add maxResultBuffer property. 
(*) Contribution by Marcin Szymborski (Github marszymb, mysql-connector-j/pull/50#issuecomment-668043714): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

Contribution: git_patch_445511136.txt (text/plain), 54.24 KiB.

[3 Aug 2020 15:12] OCA Admin
Contribution submitted via Github - Add maxResultBuffer property. 
(*) Contribution by Marcin Szymborski (Github marszymb, mysql-connector-j/pull/51#issuecomment-668043459): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

Contribution: git_patch_446754699.txt (text/plain), 37.52 KiB.

[3 Aug 2020 19:41] MySQL Verification Team
Thank you for the contribution.
[31 Aug 2020 13:47] Marcin Szymborski
Hello,

Do you any comments/feedback to my contribution?
[13 Nov 2020 15:56] Filipe Silva
Hi Marcin,

Just a heads up. We haven't had time to analyze your contribution thoroughly yet. Sorry for the delay. We hope to do it soon.

Thank you,
[16 Apr 2021 13:52] Alexander Soklakov
Hi Marcin,

Isn't it possible to just use a streaming or cursor-based result set as described on https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-reference-implementation-notes.ht... ?

It's an application design responsibility to care about memory usage. It should be clear by design what amount of data is processed, and how to do that without critical affects for the memory.

If we limit memory usage on a driver side as you propose, it would only mean that user won't be able to retrieve the data, with no workaround.
[17 May 2021 22:48] Filipe Silva
Posted by developer:
 
The patch http://github.com/mysql/mysql-connector-j/pull/51, for Connector/J 5.1, has been rejected because this series is EOL now.