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
[pdo_firebird] Added pdo_firebird_check_liveness
#12757
Conversation
28d8d6d
to
e34305d
Compare
6f5e79d
to
7cb0aba
Compare
7cb0aba
to
5d1331a
Compare
pdo_firebird_check_liveness
#if FB_API_VER >= 30 | ||
/* called by PDO to check liveness */ | ||
static zend_result pdo_firebird_check_liveness(pdo_dbh_t *dbh) /* {{{ */ | ||
{ | ||
pdo_firebird_db_handle *H = (pdo_firebird_db_handle *)dbh->driver_data; | ||
|
||
/* fb_ping return 0 if the connection is alive */ | ||
return fb_ping(H->isc_status, &H->db) ? FAILURE : SUCCESS; | ||
} | ||
/* }}} */ | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I downloaded the source from the following list and checked the included ibase.h
:
https://firebirdsql.org/en/discontinued-versions/
It turns out that this feature was introduced since v3.0.
@Girgias |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :) Maybe an upgrading entry?
Thank you for checking! Please wait a moment. |
Added to UPGRADING! |
Thank you! |
Added liveness check function for persistent connections.
todo: