Last modified: 2012-08-04 20:48:59 UTC
Created attachment 7610 [details]
10 lines changed/added in includes/db/DatabasePostgres.php
We are running a number of instances of mediawiki on PostgreSQL and are getting a large amount of warning messages in the postgres log, which could easily be fixed with a few simple changes to DatabasePostgres.php. The messages in question are the following:
Message about escape string syntax:
[2010-07-01 00:02:30.022 CEST] [dbpg-bloggwiki] [mediawiki:mediawiki] WARNING: nonstandard use of \\ in a string literal at character 25
[2010-07-01 00:02:30.022 CEST] [dbpg-bloggwiki] [mediawiki:mediawiki] HINT: Use the escape string syntax for backslashes, e.g., E'\\'.
And warning message when calling "COMMIT" when no transaction is in progress
[2010-07-01 00:10:08.521 CEST] [dbpg-bloggwiki] [mediawiki:mediawiki] WARNING: there is no transaction in progress
It's not that it's a critical problem, but it would be nice to avoid filling up our logs with 1-2GB of these kind of messages per month, in addition to it being more correct.
The checks for $mTrxLvl before COMMIT/ROLLBACK is done as of r70558.
Left off the escape thingie for \\ because I have no clue what's going on there. Somebody else can review that bit :)
This section of the postgres documentation gives a little more detail on the escape string syntax:
You can add this to your postgresql.conf to turn off those escape warnings:
escape_string_warning = off
Closing: please reopen if you see other verbosity in the logs.