diff --git a/ext/pgsql/pgsql.c b/ext/pgsql/pgsql.c index 083cd25eaad..7e94e2045b5 100644 --- a/ext/pgsql/pgsql.c +++ b/ext/pgsql/pgsql.c @@ -3593,9 +3593,6 @@ PHP_FUNCTION(pg_send_query) char *query; size_t len; PGconn *pgsql; -#ifdef LIBPQ_HAS_PIPELINING - bool is_pipeline_mode; -#endif int is_non_blocking; int ret; @@ -3607,40 +3604,23 @@ PHP_FUNCTION(pg_send_query) CHECK_PGSQL_LINK(link); pgsql = link->conn; -#ifdef LIBPQ_HAS_PIPELINING - is_pipeline_mode = (PQpipelineStatus(pgsql) == PQ_PIPELINE_ON); - if (is_pipeline_mode) { - is_non_blocking = 1; - } else { -#endif - is_non_blocking = PQisnonblocking(pgsql); + is_non_blocking = PQisnonblocking(pgsql); - if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { - php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); - RETURN_FALSE; - } - - if (_php_pgsql_link_has_results(pgsql)) { - php_error_docref(NULL, E_NOTICE, - "There are results on this connection. Call pg_get_result() until it returns FALSE"); - } -#ifdef LIBPQ_HAS_PIPELINING + if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { + php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); + RETURN_FALSE; + } + + if (_php_pgsql_link_has_results(pgsql)) { + php_error_docref(NULL, E_NOTICE, + "There are results on this connection. Call pg_get_result() until it returns FALSE"); } -#endif if (is_non_blocking) { if (!PQsendQuery(pgsql, query)) { RETURN_FALSE; } -#ifdef LIBPQ_HAS_PIPELINING - if (is_pipeline_mode) { - ret = 0; - } else { -#endif - ret = PQflush(pgsql); -#ifdef LIBPQ_HAS_PIPELINING - } -#endif + ret = PQflush(pgsql); } else { if (!PQsendQuery(pgsql, query)) { if ((PGG(auto_reset_persistent) & 2) && PQstatus(pgsql) != CONNECTION_OK) { @@ -3685,9 +3665,6 @@ PHP_FUNCTION(pg_send_query_params) char *query; size_t query_len; PGconn *pgsql; -#ifdef LIBPQ_HAS_PIPELINING - bool is_pipeline_mode; -#endif int is_non_blocking; int ret; @@ -3699,26 +3676,17 @@ PHP_FUNCTION(pg_send_query_params) CHECK_PGSQL_LINK(link); pgsql = link->conn; -#ifdef LIBPQ_HAS_PIPELINING - is_pipeline_mode = (PQpipelineStatus(pgsql) == PQ_PIPELINE_ON); - if (is_pipeline_mode) { - is_non_blocking = 1; - } else { -#endif - is_non_blocking = PQisnonblocking(pgsql); + is_non_blocking = PQisnonblocking(pgsql); - if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { - php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); - RETURN_FALSE; - } - - if (_php_pgsql_link_has_results(pgsql)) { - php_error_docref(NULL, E_NOTICE, - "There are results on this connection. Call pg_get_result() until it returns FALSE"); - } -#ifdef LIBPQ_HAS_PIPELINING + if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { + php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); + RETURN_FALSE; + } + + if (_php_pgsql_link_has_results(pgsql)) { + php_error_docref(NULL, E_NOTICE, + "There are results on this connection. Call pg_get_result() until it returns FALSE"); } -#endif num_params = zend_hash_num_elements(Z_ARRVAL_P(pv_param_arr)); if (num_params > 0) { @@ -3757,15 +3725,7 @@ PHP_FUNCTION(pg_send_query_params) } if (is_non_blocking) { -#ifdef LIBPQ_HAS_PIPELINING - if (is_pipeline_mode) { - ret = 0; - } else { -#endif - ret = PQflush(pgsql); -#ifdef LIBPQ_HAS_PIPELINING - } -#endif + ret = PQflush(pgsql); } else { /* Wait to finish sending buffer */ while ((ret = PQflush(pgsql))) { @@ -3799,9 +3759,6 @@ PHP_FUNCTION(pg_send_prepare) char *query, *stmtname; size_t stmtname_len, query_len; PGconn *pgsql; -#ifdef LIBPQ_HAS_PIPELINING - bool is_pipeline_mode; -#endif int is_non_blocking; int ret; @@ -3813,26 +3770,17 @@ PHP_FUNCTION(pg_send_prepare) CHECK_PGSQL_LINK(link); pgsql = link->conn; -#ifdef LIBPQ_HAS_PIPELINING - is_pipeline_mode = (PQpipelineStatus(pgsql) == PQ_PIPELINE_ON); - if (is_pipeline_mode) { - is_non_blocking = 1; - } else { -#endif - is_non_blocking = PQisnonblocking(pgsql); + is_non_blocking = PQisnonblocking(pgsql); - if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { - php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); - RETURN_FALSE; - } - - if (_php_pgsql_link_has_results(pgsql)) { - php_error_docref(NULL, E_NOTICE, - "There are results on this connection. Call pg_get_result() until it returns FALSE"); - } -#ifdef LIBPQ_HAS_PIPELINING + if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { + php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); + RETURN_FALSE; + } + + if (_php_pgsql_link_has_results(pgsql)) { + php_error_docref(NULL, E_NOTICE, + "There are results on this connection. Call pg_get_result() until it returns FALSE"); } -#endif if (!PQsendPrepare(pgsql, stmtname, query, 0, NULL)) { if (is_non_blocking) { @@ -3848,15 +3796,7 @@ PHP_FUNCTION(pg_send_prepare) } if (is_non_blocking) { -#ifdef LIBPQ_HAS_PIPELINING - if (is_pipeline_mode) { - ret = 0; - } else { -#endif - ret = PQflush(pgsql); -#ifdef LIBPQ_HAS_PIPELINING - } -#endif + ret = PQflush(pgsql); } else { /* Wait to finish sending buffer */ while ((ret = PQflush(pgsql))) { @@ -3892,9 +3832,6 @@ PHP_FUNCTION(pg_send_execute) char *stmtname; size_t stmtname_len; PGconn *pgsql; -#ifdef LIBPQ_HAS_PIPELINING - bool is_pipeline_mode; -#endif int is_non_blocking; int ret; @@ -3906,26 +3843,17 @@ PHP_FUNCTION(pg_send_execute) CHECK_PGSQL_LINK(link); pgsql = link->conn; -#ifdef LIBPQ_HAS_PIPELINING - is_pipeline_mode = (PQpipelineStatus(pgsql) == PQ_PIPELINE_ON); - if (is_pipeline_mode) { - is_non_blocking = 1; - } else { -#endif - is_non_blocking = PQisnonblocking(pgsql); + is_non_blocking = PQisnonblocking(pgsql); - if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { - php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); - RETURN_FALSE; - } - - if (_php_pgsql_link_has_results(pgsql)) { - php_error_docref(NULL, E_NOTICE, - "There are results on this connection. Call pg_get_result() until it returns FALSE"); - } -#ifdef LIBPQ_HAS_PIPELINING + if (is_non_blocking == 0 && PQsetnonblocking(pgsql, 1) == -1) { + php_error_docref(NULL, E_NOTICE, "Cannot set connection to nonblocking mode"); + RETURN_FALSE; + } + + if (_php_pgsql_link_has_results(pgsql)) { + php_error_docref(NULL, E_NOTICE, + "There are results on this connection. Call pg_get_result() until it returns FALSE"); } -#endif num_params = zend_hash_num_elements(Z_ARRVAL_P(pv_param_arr)); if (num_params > 0) { @@ -3966,15 +3894,7 @@ PHP_FUNCTION(pg_send_execute) } if (is_non_blocking) { -#ifdef LIBPQ_HAS_PIPELINING - if (is_pipeline_mode) { - ret = 0; - } else { -#endif - ret = PQflush(pgsql); -#ifdef LIBPQ_HAS_PIPELINING - } -#endif + ret = PQflush(pgsql); } else { /* Wait to finish sending buffer */ while ((ret = PQflush(pgsql))) { @@ -5964,85 +5884,4 @@ PHP_FUNCTION(pg_select) } /* }}} */ -#ifdef LIBPQ_HAS_PIPELINING -PHP_FUNCTION(pg_enter_pipeline_mode) -{ - zval *pgsql_link; - pgsql_link_handle *pgsql_handle; - - if (zend_parse_parameters(ZEND_NUM_ARGS(), "O", &pgsql_link, pgsql_link_ce) == FAILURE) { - RETURN_THROWS(); - } - - pgsql_handle = Z_PGSQL_LINK_P(pgsql_link); - CHECK_PGSQL_LINK(pgsql_handle); - - PQsetnonblocking(pgsql_handle->conn, 1); - - RETURN_BOOL(PQenterPipelineMode(pgsql_handle->conn)); -} - -PHP_FUNCTION(pg_exit_pipeline_mode) -{ - zval *pgsql_link; - pgsql_link_handle *pgsql_handle; - - if (zend_parse_parameters(ZEND_NUM_ARGS(), "O", &pgsql_link, pgsql_link_ce) == FAILURE) { - RETURN_THROWS(); - } - - pgsql_handle = Z_PGSQL_LINK_P(pgsql_link); - CHECK_PGSQL_LINK(pgsql_handle); - - PQsetnonblocking(pgsql_handle->conn, 0); - - RETURN_BOOL(PQexitPipelineMode(pgsql_handle->conn)); -} - -PHP_FUNCTION(pg_send_flush_request) -{ - zval *pgsql_link; - pgsql_link_handle *pgsql_handle; - - if (zend_parse_parameters(ZEND_NUM_ARGS(), "O", &pgsql_link, pgsql_link_ce) == FAILURE) { - RETURN_THROWS(); - } - - pgsql_handle = Z_PGSQL_LINK_P(pgsql_link); - CHECK_PGSQL_LINK(pgsql_handle); - - RETURN_BOOL(PQsendFlushRequest(pgsql_handle->conn)); -} - -PHP_FUNCTION(pg_pipeline_sync) -{ - zval *pgsql_link; - pgsql_link_handle *pgsql_handle; - - if (zend_parse_parameters(ZEND_NUM_ARGS(), "O", &pgsql_link, pgsql_link_ce) == FAILURE) { - RETURN_THROWS(); - } - - pgsql_handle = Z_PGSQL_LINK_P(pgsql_link); - CHECK_PGSQL_LINK(pgsql_handle); - - RETURN_BOOL(PQpipelineSync(pgsql_handle->conn)); -} - -PHP_FUNCTION(pg_pipeline_status) -{ - zval *pgsql_link; - pgsql_link_handle *pgsql_handle; - - if (zend_parse_parameters(ZEND_NUM_ARGS(), "O", &pgsql_link, pgsql_link_ce) == FAILURE) { - RETURN_THROWS(); - } - - pgsql_handle = Z_PGSQL_LINK_P(pgsql_link); - CHECK_PGSQL_LINK(pgsql_handle); - - RETURN_LONG(PQpipelineStatus(pgsql_handle->conn)); -} -#endif - #endif diff --git a/ext/pgsql/pgsql.stub.php b/ext/pgsql/pgsql.stub.php index 83ba55323e2..b706959d894 100644 --- a/ext/pgsql/pgsql.stub.php +++ b/ext/pgsql/pgsql.stub.php @@ -440,29 +440,6 @@ namespace { const PGSQL_TRACE_REGRESS_MODE = UNKNOWN; #endif -#ifdef LIBPQ_HAS_PIPELINING - /** - * @var int - * @cvalue PGRES_PIPELINE_SYNC - */ - const PGSQL_PIPELINE_SYNC = UNKNOWN; - /** - * @var int - * @cvalue PQ_PIPELINE_ON - */ - const PGSQL_PIPELINE_ON = UNKNOWN; - /** - * @var int - * @cvalue PQ_PIPELINE_OFF - */ - const PGSQL_PIPELINE_OFF = UNKNOWN; - /** - * @var int - * @cvalue PQ_PIPELINE_ABORTED - */ - const PGSQL_PIPELINE_ABORTED = UNKNOWN; -#endif - #ifdef HAVE_PG_CONTEXT_VISIBILITY /* For pg_set_error_context_visibility() */ @@ -963,14 +940,6 @@ namespace { */ function pg_select(PgSql\Connection $connection, string $table_name, array $conditions = [], int $flags = PGSQL_DML_EXEC, int $mode = PGSQL_ASSOC): array|string|false {} -#ifdef LIBPQ_HAS_PIPELINING - function pg_enter_pipeline_mode(PgSql\Connection $connection): bool {} - function pg_exit_pipeline_mode(PgSql\Connection $connection): bool {} - function pg_send_flush_request(PgSql\Connection $connection): bool {} - function pg_pipeline_sync(PgSql\Connection $connection): bool {} - function pg_pipeline_status(PgSql\Connection $connection): int {} -#endif - #ifdef HAVE_PG_CONTEXT_VISIBILITY function pg_set_error_context_visibility(PgSql\Connection $connection, int $visibility): int {} #endif diff --git a/ext/pgsql/pgsql_arginfo.h b/ext/pgsql/pgsql_arginfo.h index f7d7cc727b1..8e1fa794667 100644 --- a/ext/pgsql/pgsql_arginfo.h +++ b/ext/pgsql/pgsql_arginfo.h @@ -1,5 +1,5 @@ /* This is a generated file, edit the .stub.php file instead. - * Stub hash: b52f6b94faabab12457bf0d8ec035295aaacbfba */ + * Stub hash: a37b9df0c3b172d1160b1a7ef953cbd5a0a811b6 */ ZEND_BEGIN_ARG_WITH_RETURN_OBJ_TYPE_MASK_EX(arginfo_pg_connect, 0, 1, PgSql\\Connection, MAY_BE_FALSE) ZEND_ARG_TYPE_INFO(0, connection_string, IS_STRING, 0) @@ -452,30 +452,6 @@ ZEND_BEGIN_ARG_WITH_RETURN_TYPE_MASK_EX(arginfo_pg_select, 0, 2, MAY_BE_ARRAY|MA ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, mode, IS_LONG, 0, "PGSQL_ASSOC") ZEND_END_ARG_INFO() -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_pg_enter_pipeline_mode, 0, 1, _IS_BOOL, 0) - ZEND_ARG_OBJ_INFO(0, connection, PgSql\\Connection, 0) -ZEND_END_ARG_INFO() -#endif - -#if defined(LIBPQ_HAS_PIPELINING) -#define arginfo_pg_exit_pipeline_mode arginfo_pg_enter_pipeline_mode -#endif - -#if defined(LIBPQ_HAS_PIPELINING) -#define arginfo_pg_send_flush_request arginfo_pg_enter_pipeline_mode -#endif - -#if defined(LIBPQ_HAS_PIPELINING) -#define arginfo_pg_pipeline_sync arginfo_pg_enter_pipeline_mode -#endif - -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_pg_pipeline_status, 0, 1, IS_LONG, 0) - ZEND_ARG_OBJ_INFO(0, connection, PgSql\\Connection, 0) -ZEND_END_ARG_INFO() -#endif - #if defined(HAVE_PG_CONTEXT_VISIBILITY) ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_pg_set_error_context_visibility, 0, 2, IS_LONG, 0) ZEND_ARG_OBJ_INFO(0, connection, PgSql\\Connection, 0) @@ -575,21 +551,6 @@ ZEND_FUNCTION(pg_insert); ZEND_FUNCTION(pg_update); ZEND_FUNCTION(pg_delete); ZEND_FUNCTION(pg_select); -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_FUNCTION(pg_enter_pipeline_mode); -#endif -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_FUNCTION(pg_exit_pipeline_mode); -#endif -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_FUNCTION(pg_send_flush_request); -#endif -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_FUNCTION(pg_pipeline_sync); -#endif -#if defined(LIBPQ_HAS_PIPELINING) -ZEND_FUNCTION(pg_pipeline_status); -#endif #if defined(HAVE_PG_CONTEXT_VISIBILITY) ZEND_FUNCTION(pg_set_error_context_visibility); #endif @@ -710,21 +671,6 @@ static const zend_function_entry ext_functions[] = { ZEND_FE(pg_update, arginfo_pg_update) ZEND_FE(pg_delete, arginfo_pg_delete) ZEND_FE(pg_select, arginfo_pg_select) -#if defined(LIBPQ_HAS_PIPELINING) - ZEND_FE(pg_enter_pipeline_mode, arginfo_pg_enter_pipeline_mode) -#endif -#if defined(LIBPQ_HAS_PIPELINING) - ZEND_FE(pg_exit_pipeline_mode, arginfo_pg_exit_pipeline_mode) -#endif -#if defined(LIBPQ_HAS_PIPELINING) - ZEND_FE(pg_send_flush_request, arginfo_pg_send_flush_request) -#endif -#if defined(LIBPQ_HAS_PIPELINING) - ZEND_FE(pg_pipeline_sync, arginfo_pg_pipeline_sync) -#endif -#if defined(LIBPQ_HAS_PIPELINING) - ZEND_FE(pg_pipeline_status, arginfo_pg_pipeline_status) -#endif #if defined(HAVE_PG_CONTEXT_VISIBILITY) ZEND_FE(pg_set_error_context_visibility, arginfo_pg_set_error_context_visibility) #endif @@ -848,18 +794,6 @@ static void register_pgsql_symbols(int module_number) #if defined(PQTRACE_REGRESS_MODE) REGISTER_LONG_CONSTANT("PGSQL_TRACE_REGRESS_MODE", PQTRACE_REGRESS_MODE, CONST_PERSISTENT); #endif -#if defined(LIBPQ_HAS_PIPELINING) - REGISTER_LONG_CONSTANT("PGSQL_PIPELINE_SYNC", PGRES_PIPELINE_SYNC, CONST_PERSISTENT); -#endif -#if defined(LIBPQ_HAS_PIPELINING) - REGISTER_LONG_CONSTANT("PGSQL_PIPELINE_ON", PQ_PIPELINE_ON, CONST_PERSISTENT); -#endif -#if defined(LIBPQ_HAS_PIPELINING) - REGISTER_LONG_CONSTANT("PGSQL_PIPELINE_OFF", PQ_PIPELINE_OFF, CONST_PERSISTENT); -#endif -#if defined(LIBPQ_HAS_PIPELINING) - REGISTER_LONG_CONSTANT("PGSQL_PIPELINE_ABORTED", PQ_PIPELINE_ABORTED, CONST_PERSISTENT); -#endif #if defined(HAVE_PG_CONTEXT_VISIBILITY) REGISTER_LONG_CONSTANT("PGSQL_SHOW_CONTEXT_NEVER", PQSHOW_CONTEXT_NEVER, CONST_PERSISTENT); #endif diff --git a/ext/pgsql/tests/pg_pipeline_sync.phpt b/ext/pgsql/tests/pg_pipeline_sync.phpt deleted file mode 100644 index d3d43469b41..00000000000 --- a/ext/pgsql/tests/pg_pipeline_sync.phpt +++ /dev/null @@ -1,130 +0,0 @@ ---TEST-- -PostgreSQL pipeline mode ---EXTENSIONS-- -pgsql ---SKIPIF-- - ---FILE-- - ---EXPECT-- -OK