B4J Question Downside to not closing MySQL connections?

G

GuyBooth

Active Member
Licensed User
Longtime User
I open a SQL connection each time a make an async SQL call or sometimes a group of async calls. Each time I open a connection, I try to close it after the call is complete, but there are occasions when this results in an error where I am trying to run a SQL request against a closed database.
B4X: 
java.sql.SQLException: execute() is called on closed connection
Is there a downside to not closing each open connection? I would still open a connection for each call to make sure I have a valid connection.
 
Sort by date Sort by votes
G

GuyBooth

Active Member
Licensed User
Longtime User
Erel said:
I recommend you to use jConnectionPool. This way you will not need to manage the connections and you will always get a valid connection.
Click to expand...
Ok … is there any documentation for using jConnectionPool? Or examples? I couldn't find any, perhaps I am looking in the wrong place...
I have got as far as initializing with
B4X: 
    CP.Initialize(driver, jdbcUrl, Username, Password)
However my next step
B4X: 
    CP.GetConnectionAsync("sqlMB")
    wait for sqlMB_ConnectionReady(Success As Boolean, sqlMB As SQL)
throws a compiler error "Parameter name cannot hide global variable name"
My sqlMB is the global name for the SQL I'm using throughout the application, so I'm not sure what the connection pool is looking for here.

My original question also still stands. The jConnectionPool.getConnection help bubble says "Make sure to close the connection when done with it" but the jConnectionPool.getConnectionAsync does not. Do I need to close each connection when I use jConnectionPool.getConnectionAsync? What happens if I don't?
 
Upvote 0
G

GuyBooth

Active Member
Licensed User
Longtime User
DonManfred said:
just change the name in the wait for line...
B4X: 
wait for sqlMB_ConnectionReady(Success As Boolean, sql As SQL)
Click to expand...
What do I do with the SQL returned - ignore it?
[Edit] No that doesn't work. This works:
B4X: 
sqlMB = sql_MB
'or when returning (Success, SQL)
sqlMB = SQL

but I'm not sure why I'm doing it.​
 
Last edited:
Upvote 0
G

GuyBooth

Active Member
Licensed User
Longtime User
Erel said:
Don't have a global SQL variable.
Work with the local one and close it when done.

This way you let the connection pool take care of managing the remote connections.
Click to expand...
OK, got it. Thank you.
 
Upvote 0
You must log in or register to reply here.

Similar Threads

  • Article
Share My Creation [Web][SithasoDaisy] Revolutionize Your Ride: The Ultimate Vehicle Expense Tracker App using Api Key Secure MySQL REST API on top of HTTPS
Replies
9
Views
4K
Mashiane
  • Article
Share My Creation [Web][SithasoDaisy] Secure CRUD MySQL REST API Web App using Api Key on top of HTTPS
Replies
0
Views
2K
Mashiane
  • Article
Share My Creation [Project Template] Web API Server v1
4 5 6
Replies
111
Views
65K
PABLO2013
  • Article
B4J Library [B4X] [B4XLib] SD_SQL (direct access to MySQL, MariaDB, MS SQL, FireBird)
2 3 4
Replies
68
Views
40K
Mashiane
  • Article
Share My Creation Erels [B4X] CloudKVS - synchronized key / value store - MySQL
Replies
12
Views
13K
janderkan
Menu
Log in
Register
Cookies are required to use this site. You must accept them to continue using the site. Learn more…