From 5f650aef58fb97cfee1106e44802cc8ff7fcb65d Mon Sep 17 00:00:00 2001 From: Louis Lam Date: Thu, 16 Oct 2025 07:30:26 +0800 Subject: [PATCH] Fix: Set default value for `ping` column to 0 in stat tables (#6188) --- .../2025-10-15-0000-stat-table-fix.js | 27 +++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 db/knex_migrations/2025-10-15-0000-stat-table-fix.js diff --git a/db/knex_migrations/2025-10-15-0000-stat-table-fix.js b/db/knex_migrations/2025-10-15-0000-stat-table-fix.js new file mode 100644 index 000000000..7c3038692 --- /dev/null +++ b/db/knex_migrations/2025-10-15-0000-stat-table-fix.js @@ -0,0 +1,27 @@ +// Fix for #4315. Logically, setting it to 0 ping may not be correct, but it is better than throwing errors + +exports.up = function (knex) { + return knex.schema + .alterTable("stat_daily", function (table) { + table.integer("ping").defaultTo(0).alter(); + }) + .alterTable("stat_hourly", function (table) { + table.integer("ping").defaultTo(0).alter(); + }) + .alterTable("stat_minutely", function (table) { + table.integer("ping").defaultTo(0).alter(); + }); +}; + +exports.down = function (knex) { + return knex.schema + .alterTable("stat_daily", function (table) { + table.integer("ping").alter(); + }) + .alterTable("stat_hourly", function (table) { + table.integer("ping").alter(); + }) + .alterTable("stat_minutely", function (table) { + table.integer("ping").alter(); + }); +};