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: http://www.postgresql.org/docs/8.4/static/sql-syntax-lexical.html#SQL-SYNTAX-STRINGS-ESCAPE
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.