Tarantool development patches archive
 help / color / mirror / Atom feed
* [Tarantool-patches] [PATCH] sql: increase default cache size
@ 2020-12-10 22:37 Leonid Vasiliev
  2020-12-11 12:09 ` Nikita Pettik
  2020-12-11 14:12 ` Sergey Ostanevich
  0 siblings, 2 replies; 9+ messages in thread
From: Leonid Vasiliev @ 2020-12-10 22:37 UTC (permalink / raw)
  To: v.shpilevoy, imeevma, korablev, sergos; +Cc: tarantool-patches

Increase the maximum number of in-memory pages to use
for temporary tables.
(https://www.sqlite.org/compile.html#default_cache_size)

Part of #5609
---

Hi SQL team. It is a simplest part of #5609.
This patch increases the default SQL cache size by 10 times to 20 MB.
A similar experiment shows a 10% performance increase for some datasets.
(https://github.com/tarantool/tarantool/issues/5593#issuecomment-740044007).
Mons approved these changes. I haven't done any research to determine the
optimal value. So feel free to throw this patch away.

https://github.com/tarantool/tarantool/issues/5609
https://github.com/tarantool/tarantool/tree/lvasiliev/gh-5609-increase-default-sql-sort-cache-size

 src/box/CMakeLists.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/box/CMakeLists.txt b/src/box/CMakeLists.txt
index 19203f7..7372179 100644
--- a/src/box/CMakeLists.txt
+++ b/src/box/CMakeLists.txt
@@ -218,6 +218,10 @@ if(CMAKE_BUILD_TYPE STREQUAL "Debug")
   add_definitions(-DSQL_DEBUG=1)
 endif()
 add_definitions(-DSQL_TEST=1)
+# Set the maximum number of in-memory pages to use for temporary tables.
+# 20000 * 1024 = 20480000 bytes.
+# (https://www.sqlite.org/compile.html#default_cache_size)
+add_definitions(-DSQL_DEFAULT_CACHE_SIZE=-20000)
 
 set(EXT_SRC_DIR ${CMAKE_SOURCE_DIR}/extra)
 set(EXT_BIN_DIR ${CMAKE_BINARY_DIR}/extra)
-- 
2.7.4

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-12-30 13:03 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-10 22:37 [Tarantool-patches] [PATCH] sql: increase default cache size Leonid Vasiliev
2020-12-11 12:09 ` Nikita Pettik
2020-12-11 15:29   ` Leonid Vasiliev
2020-12-24 16:57     ` Nikita Pettik
2020-12-28  8:54       ` Leonid Vasiliev
2020-12-28 12:13         ` Nikita Pettik
2020-12-30 13:02           ` Leonid Vasiliev
2020-12-11 14:12 ` Sergey Ostanevich
2020-12-11 15:24   ` Leonid Vasiliev

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox