Skip to content

Conversation

@saaj
Copy link
Contributor

@saaj saaj commented Jul 29, 2025

Upgrades:

  • SQLite 3.41.0 → 3.50.3
  • SQLite miscellaneous extensions to latest versions
  • pivot_vtab extension: to its master commit (fix segfault)
  • Docker image for running tests in CI: ubuntu-20.04 → 24.04, switch to Firefox ESR from Mozilla's PPA (otherwise Karma hangs)
  • Dockerfile.test: node:12.22-buster → node:12.22-bullseye (Buster repos are dead)

Behaviour change:

WITH input(filename) AS (
  VALUES
    ('/etc/redis/redis.conf'),
    ('/run/redis/redis-server.pid'),
    ('/var/log/redis-server.log')
), tmp AS (
  SELECT
    filename,
    '["' || replace(filename, '/', '", "') || '"]' as filename_array
  FROM input
)
SELECT (
  SELECT group_concat(ip.value, '/')
  FROM json_each(filename_array) ip
  WHERE ip.id <= p.id
) AS path
FROM tmp, json_each(filename_array) AS p
WHERE p.id > 1  -- because the filenames start with the separator

Return empty string for each path, since SQLite 3.47:

Fix the group_concat() aggregate function so that it returns an empty string, not a NULL, if it receives a single input value which is an empty string.

s/WHERE p.id > 1/WHERE p.key > 0/ makes the test query consistent with expected output.

result-analysis.pdf

@saaj saaj force-pushed the update_sqlite_to_3_50 branch 2 times, most recently from 755b199 to 937a09b Compare July 30, 2025 21:12
@saaj saaj force-pushed the update_sqlite_to_3_50 branch from 937a09b to a9d6167 Compare July 30, 2025 21:15
@lana-k
Copy link
Owner

lana-k commented Jul 30, 2025

#125

@lana-k lana-k merged commit 75bf849 into lana-k:master Jul 30, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants