From a9c4e2da279729f708293ceab2ae2a742ceb6c39 Mon Sep 17 00:00:00 2001 From: Mehdi Bouaziz Date: Wed, 27 May 2026 16:24:04 +0000 Subject: [PATCH] runtime: drop debug stdout print when SKIP_CAPACITY is set MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit sk_create_mapping printed "CAPACITY SET TO: " to stdout for every Skip process started with a non-default capacity. This was debug output that escaped into normal stdout — fine when nothing relied on stdout exactness, but the skdb diff tests (test/diff/*.sql) compare runtime stdout to .expected golden files and now fail on every test once SKIP_CAPACITY is set in CI (e.g. via the gen2 fix in #1246). Just remove the print. The value is observable via /proc/self/maps or by recognising the cgroup limit; no observability is being lost that needed to be on stdout. Co-Authored-By: Claude Opus 4.7 (1M context) --- skiplang/prelude/runtime/palloc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/skiplang/prelude/runtime/palloc.c b/skiplang/prelude/runtime/palloc.c index e43cbfb39..477ab8f0e 100644 --- a/skiplang/prelude/runtime/palloc.c +++ b/skiplang/prelude/runtime/palloc.c @@ -564,9 +564,6 @@ void sk_create_mapping(char* fileName, size_t icapacity) { ginfo->fileName = (fileName != NULL) ? persistent_fileName : NULL; ginfo->contexts = NULL; *gid = 1; - if (icapacity != DEFAULT_CAPACITY) { - printf("CAPACITY SET TO: %ld\n", icapacity); - } *capacity = icapacity; *pconsts = NULL;