Page MenuHomePhabricator

MobileFrontend doesn't support PostgreSQL
Closed, ResolvedPublic

Description

For example, mobile watchlist query fails.


Version: unspecified
Severity: enhancement

Details

Reference
bz46745

Related Objects

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
InvalidNone
ResolvedNone

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:32 AM
bzimport set Reference to bz46745.
bzimport added a subscriber: Unknown Object (MLST).
MaxSem created this task.Mar 31 2013, 11:50 PM

Status? Do we really want to fix this?

Regarding "Do we really want to fix this", is that a philosophical question or a pragmatic one? Using select "table.*" seems like generally bad practice, and in this case should be easy enough to fix (provided that wl_notificationtimestamp is not actually used, which it seems not to be). But will it turn into a never ending game of whack-a-mole?

(In reply to Jon from comment #2)

Status? Do we really want to fix this?

And who is "we"? Mobile team? Community? If mobile team wouldn't want to fix it, would it still accept and review contributed patches to fix this?

It's an open question. We can be anyone. I don't know how much work is involved in this and I wasn't sure who cared about this and there has been no activity on the bug since it was opened a year ago. Now we have 2 comments so I suceeded in my goal )

At the risk of speaking for the WMF mobile team's product owner, this is not something the mobile team will work on - at least not in an organized/prioritized/etc way - any time soon (we may need to revisit this if/when mobile functionality gets moved to core). In the meantime, if anyones wants to pick this up and work on it otherwise, I think that would be swell - after all, MediaWiki core is compatible with Postgres (https://www.mediawiki.org/wiki/Manual:Installation_requirements).

Change 132586 had a related patch set uploaded by Jjanes:
Fix watchlist query to work with PostgreSQL

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

Change 132586 merged by jenkins-bot:
Fix watchlist query to work with PostgreSQL

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

Jdforrester-WMF added a subscriber: Jdforrester-WMF.

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