Skip to content
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

Performance degrades with larger result sets. #13

Closed
abmusse opened this issue Nov 16, 2017 · 3 comments
Closed

Performance degrades with larger result sets. #13

abmusse opened this issue Nov 16, 2017 · 3 comments
Labels
blocker bug Something isn't working

Comments

@abmusse
Copy link
Member

abmusse commented Nov 16, 2017

Original report by Kristopher Baehr (Bitbucket: krisbaehr, GitHub: krisbaehr).


Whether running SQL statements, or SP calls, performance degrades significantly with this connector as the number of rows being returned increases. We have a stored procedure that returns a user-specified number of rows. When calling the same Stored Procedure with Node.js and JDBC on Tomcat, 100 rows or fewer results in Node.js is faster. But, when calling with 1000 rows, Node.js response times were averaging 650 ms, and JDBC on Tomcat average 250 ms. Nearly all time for these https requests is spent on the stmt.execute() and stmt.fetchAll() for the stored procedure call. The performance monitor indicated very similar response times from the SQL engine between these two methods. This indicates that something may need to be tweaked in the connector, or perhaps there is a connection or statement attribute that I need to set. Please advise.

@abmusse
Copy link
Member Author

abmusse commented Nov 17, 2017

Original comment by Kristopher Baehr (Bitbucket: krisbaehr, GitHub: krisbaehr).


Upgrading to blocker. We cannot use Node.js on the IBM i for production purposes until this is resolved.

@abmusse
Copy link
Member Author

abmusse commented Nov 21, 2017

Original comment by Kristopher Baehr (Bitbucket: krisbaehr, GitHub: krisbaehr).


I've been performance testing to get some more concrete numbers and from what I'm seeing this connector is not the issue. This issue can be closed.

@abmusse
Copy link
Member Author

abmusse commented Nov 21, 2017

Original comment by Kristopher Baehr (Bitbucket: krisbaehr, GitHub: krisbaehr).


Not an issue with the connector.

@abmusse abmusse closed this as completed Nov 21, 2017
@abmusse abmusse added blocker bug Something isn't working labels Jan 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant