Page MenuHomePhabricator

Quotes needed for timestamps in PostgreSQL
Closed, ResolvedPublic

Description

The fix for bug T114806 doesn't quote timestamps it sends directly to the database (i.e. not in bind variables).

For reasons I don't understand, this causes 'php phpunit.php includes/ImportLinkCacheIntegrationTest.php' to blow up spectacularly when used against PostgreSQL, consuming all memory and crashing.

A simple addQuotes fixes that problem, but it would be nice to fix the problem with phpunit that causes infinite loops with memory allocation, too.

Event Timeline

Jjanes raised the priority of this task from to Needs Triage.
Jjanes updated the task description. (Show Details)
Jjanes added a project: MediaWiki-libs-Rdbms.
Jjanes subscribed.

Change 259636 had a related patch set uploaded (by Jjanes):
PostgreSQL: Add quotes to timestamp

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

Change 259636 merged by jenkins-bot:
PostgreSQL: Add quotes to timestamp

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

Paladox assigned this task to Jjanes.