New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slow read of blob and high memory usage [JDBC497] #535
Comments
Modified by: A Drouard (adrouard)Attachment: TestReadBlob.java [ 13130 ] |
Commented by: @mrotteveel What is the size of the blobs on average? |
Commented by: A Drouard (adrouard) The size of the blobs on average is 200 bytes. |
Commented by: @mrotteveel It looks like Jaybird is not removing the blob handles from databaseListenerDispatcher after blob close. |
Commented by: @mrotteveel The problem was indeed a leak of blob handles. Snapshot with fix on Maven Sonatype snapshot repository as 3.0.1-SNAPSHOT Download: I'm considering to release 3.0.1 soon. |
Commented by: @mrotteveel Clarification: "Soon" means somewhere next week (probably Monday) |
Commented by: @mrotteveel Released with Jaybird 3.0.1 : https://github.com/FirebirdSQL/jaybird/releases/tag/v3.0.1 |
Modified by: @mrotteveelstatus: Resolved [ 5 ] => Closed [ 6 ] |
Submitted by: A Drouard (adrouard)
Attachments:
TestReadBlob.java
I recently upgraded from Jaybird 2.2.9 to Jaybird 3.0.0.
My application is reading a lot of BLOB, and since the upgrade, I noticed a degradation of performance and memory usage.
The problem is particulary visible when reading 2 BLOB columns.
-----------------------------------
Reading 1 blob column :
Jaybird 2.2.9 : 300 000 rows fetched in 45 564 ms, connection closed in 0 ms, heap memory usage : 20 MB
Jaybird 3.0.0 : 300 000 rows fetched in 77 130 ms, connection closed in 28 000 ms, heap memory usage : 687 MB
-----------------------------------
Reading 2 blob columns :
Jaybird 2.2.9 : 300 000 rows fetched in 102 960 ms, connection closed in 1 ms, heap memory usage : 25 MB
Jaybird 3.0.0 : 300 000 rows fetched in 325 930 ms, connection closed in 16 3967 ms, heap memory usage : 653 MB
Commits: 5463983 8a38eb9 5cb3b74 24cf380
The text was updated successfully, but these errors were encountered: