Skip to content

Commit 6e38c0f

Browse files
author
Vladimir Ershov
committed
fix test - reduce warnings
1 parent f834645 commit 6e38c0f

File tree

3 files changed

+3
-8
lines changed

3 files changed

+3
-8
lines changed

expected/sr_plan.out

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,19 +34,16 @@ SELECT * FROM test_table WHERE test_attr1 = 15;
3434

3535
UPDATE sr_plans SET enable = true;
3636
SELECT * FROM test_table WHERE test_attr1 = _p(10);
37-
WARNING: Ok we find saved plan.
3837
test_attr1 | test_attr2
3938
------------+------------
4039
(0 rows)
4140

4241
SELECT * FROM test_table WHERE test_attr1 = _p(15);
43-
WARNING: Ok we find saved plan.
4442
test_attr1 | test_attr2
4543
------------+------------
4644
(0 rows)
4745

4846
SELECT * FROM test_table WHERE test_attr1 = 10;
49-
WARNING: Ok we find saved plan.
5047
test_attr1 | test_attr2
5148
------------+------------
5249
(0 rows)

sr_plan--1.0.sql

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ CREATE TABLE sr_plans (
1414

1515
CREATE INDEX sr_plans_query_hash_idx ON sr_plans (query_hash);
1616
--CREATE INDEX sr_plans_plan_hash_idx ON sr_plans (plan_hashs);
17-
--create function _p(anyelement) returns anyelement as $$ select $1; $$ language sql VOLATILE;
1817

1918
CREATE FUNCTION _p(anyelement)
2019
RETURNS anyelement
@@ -32,4 +31,4 @@ CREATE FUNCTION sr_plan_invalid_table() RETURNS event_trigger
3231

3332
CREATE EVENT TRIGGER sr_plan_invalid_table ON sql_drop
3433
-- WHEN TAG IN ('DROP TABLE')
35-
EXECUTE PROCEDURE sr_plan_invalid_table();
34+
EXECUTE PROCEDURE sr_plan_invalid_table();

sr_plan.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -210,8 +210,7 @@ PlannedStmt *sr_planner(Query *parse,
210210
HeapTuple local_tuple;
211211
local_tuple = index_getnext(query_index_scan, ForwardScanDirection);
212212

213-
if (local_tuple == NULL)
214-
break;
213+
if (local_tuple == NULL) break;
215214

216215
heap_deform_tuple(local_tuple, sr_plans_heap->rd_att,
217216
search_values, search_nulls);
@@ -227,7 +226,7 @@ PlannedStmt *sr_planner(Query *parse,
227226

228227
if (find_ok)
229228
{
230-
elog(WARNING, "Ok we find saved plan.");
229+
/* elog(WARNING, "Ok we find saved plan."); */
231230
out_jsonb2 = (Jsonb *)DatumGetPointer(PG_DETOAST_DATUM(search_values[3]));
232231
if (query_params != NULL)
233232
pl_stmt = jsonb_to_node_tree(out_jsonb2, &replace_fake);

0 commit comments

Comments
 (0)