Page MenuHomePhabricator
Create Task
Maniphest T62531

Media deletion fails with database error on PostgreSQL backend
Closed, ResolvedPublic
Actions

Description

Author: gebhkla

Description:
At least on SQL query used for media deletion is incorrectly generated for PostgreSQL backends: In the FROM clause tables are missing:

SELECT rev_id,rev_page,rev_text_id,rev_timestamp,rev_comment,rev_user_text,rev_user,rev_minor_edit,rev_deleted,rev_len,rev_parent_id,rev_sha1,rev_content_format,rev_content_model,page_namespace,page_title,page_id,page_latest,page_is_redirect,page_len,user_name FROM "mwuser" WHERE page_id = '207' AND rev_id = '1239' LIMIT 1 FOR UPDATE OF page

Version: 1.22.1
Severity: normal

Details

Reference
bz60531

Related Objects
Search...

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.
StatusSubtypeAssignedTask
· · ·
InvalidNoneT2384 [REPLACED BY TAG] PostgreSQL/pgsql support (tracking)
ResolvedLethosorT62531 Media deletion fails with database error on PostgreSQL backend
· · ·

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:50 AM
• bzimport added projects: MW-1.22-release, MediaWiki-libs-Rdbms.
• bzimport set Reference to bz60531.
• bzimport created this task.Jan 28 2014, 3:30 PM
• bzimport added a comment.Jan 28 2014, 3:37 PM

gebhkla wrote:

Patch

Renaming the $table to $table_cond to avoid naming conflicts with the argument of the function solved to problem.

Attached:

mediawiki-1.22.1-patch-60531.txt1 KBDownload

gerritbot added a comment.Jan 28 2014, 4:54 PM

Change 109914 had a related patch set uploaded by Umherirrender:
Avoid variable naming conflicts in DatabasePostgres::selectSQLText

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

Umherirrender added a comment.Jan 28 2014, 4:57 PM

Reapply you change manually against current master (the is_array was added in the meantime). Thanks for the explanation/tip.

In future please try to commit direct to gerrit to make the review easier. You can also use the gerrit-patch-uploader[1], if no gerrit account exists.

[1] https://tools.wmflabs.org/gerrit-patch-uploader/

Umherirrender added a comment.Jan 28 2014, 5:16 PM

Fix for bug 47055 was backported to REL1_22, so this fix should also be backported.

gerritbot added a comment.Jan 28 2014, 5:45 PM

Change 109914 merged by jenkins-bot:
Avoid variable naming conflicts in DatabasePostgres::selectSQLText

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

Umherirrender added a comment.Jan 28 2014, 5:55 PM

successfully merged

gerritbot added a comment.Feb 2 2014, 9:02 AM

Change 110836 had a related patch set uploaded by Siebrand:
Avoid variable naming conflicts in DatabasePostgres::selectSQLText

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

gerritbot added a comment.Feb 2 2014, 7:00 PM

Change 110836 merged by jenkins-bot:
Avoid variable naming conflicts in DatabasePostgres::selectSQLText

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

siebrand added a comment.Feb 3 2014, 8:19 AM

Was backported to 1.22, and will be part of 1.22.2.

Jdforrester-WMF added a project: PostgreSQL.Nov 2 2016, 8:16 PM
Jdforrester-WMF subscribed.

Migrating from the old tracking task to a tag for PostgreSQL-related tasks.

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL