Performance updates.

Benchmarking originally showed fairly high memory usage and a slow garbage
collection schedule when creating 1000 contexts.  To avoid this, a garbage
collection cycle was put into place after each function execution.  This
solved the memory bloat, but slowed down function calls.

I decided to revisit that decision.

Memory limits were an arbitrary 256MB when the concept was put into place
in plv8 in 2019, and 512MB is a decent default with modern hardware.  It
can still be changed to 256MB with a GUC if desired.

The benchmark was able to trigger an out of memory issue, but under very
artificial conditions:

* 1000 roles, and thus quickjs contexts were being created
* Each context carries an allocation of 190kb on MacOS (untested on Linux)

Even though the roles are deleted, there is no mechanism for postgres to tell
pljs this, and those contexts essentially become zombies in the cache.

Since the benchmark is an extremely artifical (and meant specifically to
test the startup time of pljs contexts), it is unlikely that this behavior
will be able to be triggered in production.  At worst case, a memory limit
is hit, and the limit needs to be increased via the GUC.

Thus garbage collection is left to run when memory pressure exists, rather
with every function execution, and more memory can be allocated to slow
memory pressure by default.

This speeds up contex creation and function calls slightly, and speeds up
the artificial context creation significantly.

Author: JerrySievert

.gitignore

@@ -9,5 +9,6 @@ regression.out
 .depend
 docs/html
 docs/latex
+docs/xml
 pljs--*.sql
 compile_commands.json

docs/CHANGELOG.md

@@ -28,3 +28,9 @@ Released _August 20, 2025_.
 Released _August 20, 2025_.
 
 - PGXN release went out with an incorrect control file
+
+### 1.1.0
+
+- Up memory default limit to 512MB
+- Remove unnecessary include from modules.c
+- Remove extra running of GC after each execution

src/pljs.c

@@ -104,8 +104,8 @@ void pljs_guc_init(void) {
 
   DefineCustomIntVariable("pljs.memory_limit",
                           gettext_noop("Runtime limit in MBytes"),
-                          gettext_noop("The default value is 256 MB"),
-                          (int *)&configuration.memory_limit, 256, 256, 3096,
+                          gettext_noop("The default value is 512 MB"),
+                          (int *)&configuration.memory_limit, 512, 256, 3096,
                           PGC_SUSET, 0, NULL, NULL, NULL);
 
   DefineCustomStringVariable(
@@ -1043,7 +1043,7 @@ static Datum call_function(FunctionCallInfo fcinfo, pljs_context *context,
 
   JSValue ret = JS_Call(context->ctx, context->js_function, JS_UNDEFINED,
                         context->function->inargs, argv);
-  JS_RunGC(rt);
+
   SPI_finish();
 
   if (JS_IsException(ret)) {