[Tarantool-patches] [PATCH v3 05/20] sql: rename sql_finalize() to sql_stmt_finalize()
Nikita Pettik
korablev at tarantool.org
Fri Dec 20 15:47:10 MSK 2019
Let's follow unified naming rules for SQL high level API which
manipulates on statements objects. To be more precise, let's use
'sql_stmt_' prefix for interface functions operating on statement
handles.
---
src/box/bind.c | 2 +-
src/box/ck_constraint.c | 4 ++--
src/box/execute.c | 2 +-
src/box/lua/execute.c | 2 +-
src/box/sql/analyze.c | 6 +++---
src/box/sql/sqlInt.h | 2 +-
src/box/sql/vdbe.c | 4 ++--
src/box/sql/vdbeapi.c | 2 +-
8 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/box/bind.c b/src/box/bind.c
index 7eea9fcc8..bbc1f56df 100644
--- a/src/box/bind.c
+++ b/src/box/bind.c
@@ -180,7 +180,7 @@ sql_bind_column(struct sql_stmt *stmt, const struct sql_bind *p,
* Parameters are allocated within message pack,
* received from the iproto thread. IProto thread
* now is waiting for the response and it will not
- * free the packet until sql_finalize. So
+ * free the packet until sql_stmt_finalize. So
* there is no need to copy the packet and we can
* use SQL_STATIC.
*/
diff --git a/src/box/ck_constraint.c b/src/box/ck_constraint.c
index a2c66ce00..551bdd397 100644
--- a/src/box/ck_constraint.c
+++ b/src/box/ck_constraint.c
@@ -141,7 +141,7 @@ ck_constraint_program_compile(struct ck_constraint_def *ck_constraint_def,
diag_set(ClientError, ER_CREATE_CK_CONSTRAINT,
ck_constraint_def->name,
box_error_message(box_error_last()));
- sql_finalize((struct sql_stmt *) v);
+ sql_stmt_finalize((struct sql_stmt *) v);
return NULL;
}
return (struct sql_stmt *) v;
@@ -254,7 +254,7 @@ error:
void
ck_constraint_delete(struct ck_constraint *ck_constraint)
{
- sql_finalize(ck_constraint->stmt);
+ sql_stmt_finalize(ck_constraint->stmt);
ck_constraint_def_delete(ck_constraint->def);
TRASH(ck_constraint);
free(ck_constraint);
diff --git a/src/box/execute.c b/src/box/execute.c
index af66447b5..fb83e1194 100644
--- a/src/box/execute.c
+++ b/src/box/execute.c
@@ -100,7 +100,7 @@ static void
port_sql_destroy(struct port *base)
{
port_tuple_vtab.destroy(base);
- sql_finalize(((struct port_sql *)base)->stmt);
+ sql_stmt_finalize(((struct port_sql *)base)->stmt);
}
const struct port_vtab port_sql_vtab = {
diff --git a/src/box/lua/execute.c b/src/box/lua/execute.c
index ffa3d4d2e..68adacf72 100644
--- a/src/box/lua/execute.c
+++ b/src/box/lua/execute.c
@@ -217,7 +217,7 @@ lua_sql_bind_list_decode(struct lua_State *L, struct sql_bind **out_bind,
size_t size = sizeof(struct sql_bind) * bind_count;
/*
* Memory allocated here will be freed in
- * sql_finalize() or in txn_commit()/txn_rollback() if
+ * sql_stmt_finalize() or in txn_commit()/txn_rollback() if
* there is an active transaction.
*/
struct sql_bind *bind = (struct sql_bind *) region_alloc(region, size);
diff --git a/src/box/sql/analyze.c b/src/box/sql/analyze.c
index 42e2a1a2f..f74f9b358 100644
--- a/src/box/sql/analyze.c
+++ b/src/box/sql/analyze.c
@@ -1424,7 +1424,7 @@ load_stat_from_space(const char *sql_select_prepare,
current_idx_count++;
}
- rc = sql_finalize(stmt);
+ rc = sql_stmt_finalize(stmt);
if (rc)
goto finalize;
rc = sql_stmt_compile(sql_select_load, -1, NULL, &stmt, 0);
@@ -1475,7 +1475,7 @@ load_stat_from_space(const char *sql_select_prepare,
sample->sample_key = region_alloc(&fiber()->gc,
sample->key_size);
if (sample->sample_key == NULL) {
- sql_finalize(stmt);
+ sql_stmt_finalize(stmt);
rc = -1;
diag_set(OutOfMemory, sample->key_size,
"region", "sample_key");
@@ -1488,7 +1488,7 @@ load_stat_from_space(const char *sql_select_prepare,
}
stats[current_idx_count].sample_count++;
}
- rc = sql_finalize(stmt);
+ rc = sql_stmt_finalize(stmt);
if (rc == 0 && prev_index != NULL)
init_avg_eq(prev_index, &stats[current_idx_count]);
assert(current_idx_count <= index_count);
diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h
index 03deb733c..cf0b946f1 100644
--- a/src/box/sql/sqlInt.h
+++ b/src/box/sql/sqlInt.h
@@ -521,7 +521,7 @@ sql_column_value(sql_stmt *,
int iCol);
int
-sql_finalize(sql_stmt * pStmt);
+sql_stmt_finalize(sql_stmt * pStmt);
/*
* Terminate the current execution of an SQL statement and reset
diff --git a/src/box/sql/vdbe.c b/src/box/sql/vdbe.c
index ab86be9a9..336fd4a52 100644
--- a/src/box/sql/vdbe.c
+++ b/src/box/sql/vdbe.c
@@ -1084,7 +1084,7 @@ case OP_Yield: { /* in1, jump */
* automatically.
*
* P1 is the result code returned by sql_exec(),
- * sql_reset(), or sql_finalize(). For a normal halt,
+ * sql_reset(), or sql_stmt_finalize(). For a normal halt,
* this should be 0.
* For errors, it can be some other value. If P1!=0 then P2 will
* determine whether or not to rollback the current transaction.
@@ -2887,7 +2887,7 @@ case OP_MakeRecord: {
* memory shouldn't be reused until it is written into WAL.
*
* However, if memory for ephemeral space is allocated
- * on region, it will be freed only in sql_finalize()
+ * on region, it will be freed only in sql_stmt_finalize()
* routine.
*/
if (bIsEphemeral) {
diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c
index ab8441bc5..7463fb9d7 100644
--- a/src/box/sql/vdbeapi.c
+++ b/src/box/sql/vdbeapi.c
@@ -76,7 +76,7 @@ invokeProfileCallback(sql * db, Vdbe * p)
* machine.
*/
int
-sql_finalize(sql_stmt * pStmt)
+sql_stmt_finalize(sql_stmt * pStmt)
{
if (pStmt == NULL)
return 0;
--
2.15.1
More information about the Tarantool-patches
mailing list