Last modified: 2013-10-15 22:54:14 UTC
Created attachment 13484 [details] proposed patch In maintenance/postgres/tables.sql, the profiling table should be defined like this: CREATE TABLE profiling ( pf_count INTEGER NOT NULL DEFAULT 0, pf_time FLOAT NOT NULL DEFAULT 0, pf_memory FLOAT NOT NULL DEFAULT 0, pf_name TEXT NOT NULL, pf_server TEXT NULL ); The current use of NUMERIC(18,10) very rapidly overflows the pf_memory column, generating errors. Also, the NUMERIC is very much slower than float, and in this case it has no advantages. Proposed patch attached.
Change 89995 had a related patch set uploaded by Gerrit Patch Uploader: In maintenance/postgres/tables.sql, the profiling table should be defined like this: https://gerrit.wikimedia.org/r/89995
Change 89995 merged by jenkins-bot: In maintenance/postgres/tables.sql, the profiling table should be defined like this: https://gerrit.wikimedia.org/r/89995