You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Background:
As part of my research in to other memory leaks, my eye fell on the space consumed by the query plan cache. I instrumented it a bit and found that the hit rate for our usage was not worth the hundreds of MB occupied by the cache, so have disabled it entirely in some of our clients. We have one query in one client for which plan caching would be useful, so I was trying to use a prepared statement to selectively cache just that one case.
Date: 2016-09-23 18:31:01 +0200
From: Richard Hughes <<richard.monetdb>>
To: SQL devs <>
Version: 11.23.7 (Jun2016-SP1)
CC: @njnes
Last updated: 2016-10-13 10:04:18 +0200
Comment 22386
Date: 2016-09-23 18:31:01 +0200
From: Richard Hughes <<richard.monetdb>>
Build is Jun2016 f32bf50af59a
To reproduce:
sql>set cache=0;
sql>prepare select 1;
Expected:
execute prepared statement using: EXEC 3(...)
+---------+--------+-------+--------+-------+--------+
| type | digits | scale | schema | table | column |
+=========+========+=======+========+=======+========+
| tinyint | 8 | 0 | | L | |
+---------+--------+-------+--------+-------+--------+
Actual:
[no output]
This seems to fix it:
diff -r c94cdb95264d sql/backends/monet5/sql_scenario.c
--- a/sql/backends/monet5/sql_scenario.c Thu Sep 15 11:09:35 2016 +0100
+++ b/sql/backends/monet5/sql_scenario.c Fri Sep 23 16:52:35 2016 +0100
@@ -1241,7 +1241,7 @@
SQLsetTrace(be, c, TRUE);
if (m->emod & mod_debug)
SQLsetDebugger(c, m, TRUE);
Background:
As part of my research in to other memory leaks, my eye fell on the space consumed by the query plan cache. I instrumented it a bit and found that the hit rate for our usage was not worth the hundreds of MB occupied by the cache, so have disabled it entirely in some of our clients. We have one query in one client for which plan caching would be useful, so I was trying to use a prepared statement to selectively cache just that one case.
Comment 22390
Date: 2016-09-29 12:48:51 +0200
From: @njnes
added the patch. Thanks for looking so deeply into the code.
Comment 22391
Date: 2016-09-29 12:50:08 +0200
From: MonetDB Mercurial Repository <>
Changeset 4f8d5096c98a made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.
For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=4f8d5096c98a
Changeset description:
Comment 24496
Date: 2016-10-13 10:04:18 +0200
From: @sjoerdmullender
Jun2016-SP2 has been released.
The text was updated successfully, but these errors were encountered: