Page MenuHomePhabricator

Increase log level of rowsAffected > 1000 on database writes in transaction profiler to warning or error
Open, MediumPublic

Event Timeline

Unfortunately we need more clean ups before turning this on: https://logstash.wikimedia.org/goto/96f4a2de181b873b95b7a78bbe96151b

I will split it to per issue and create a subticket for each.

That's a bit different. The rowsAffected > 10,000 rows are for read queries but I'm talking about write queries here. Write queries are more important because they could cause read-only as the mentioned ticket did. I can create another ticket for 10K read rows for future clean up.

Umherirrender renamed this task from Increase log level of rowsAffected > 1000 in transaction profiler to warning or error to Increase log level of rowsAffected > 1000 on database writes in transaction profiler to warning or error.Feb 23 2022, 11:08 PM

Change 766564 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[mediawiki/core@master] block: Add a limit to DatabaseBlockStore::purgeExpiredBlocks

https://gerrit.wikimedia.org/r/766564

Change 766564 merged by jenkins-bot:

[mediawiki/core@master] block: Add a limit to DatabaseBlockStore::purgeExpiredBlocks

https://gerrit.wikimedia.org/r/766564

Change 766850 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/core@master] block: Use $wgUpdateRowsPerQuery for purgeExpiredBlocks()

https://gerrit.wikimedia.org/r/766850

Change 766850 merged by jenkins-bot:

[mediawiki/core@master] block: Use $wgUpdateRowsPerQuery for purgeExpiredBlocks()

https://gerrit.wikimedia.org/r/766850