Page MenuHomePhabricator

Maintenance script for populating pp_sortkey
Open, MediumPublic

Description

Follow-up for T60032: a maintenance script for populating pp_sortkey. This will be sloppy, because when reading pp_value from the database, we don't know the original type - the value will always be a string.
We should also double-check if there is not already something similar.

Details

Reference
bz64949
Related Gerrit Patches:

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:21 AM
bzimport added a project: Wikimedia-Rdbms.
bzimport set Reference to bz64949.
bzimport added a subscriber: Unknown Object (MLST).

Could also be covered by a null-edit script.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 26 2016, 12:43 AM
MZMcBride set Security to None.

I am new here and was wondering if I could contribute to resolve this bug.
What I understood from the conversation was to create a script that populates pp_sortkey column of page_props table.
Please tell what exactly should the script do ?

Aklapper updated the task description. (Show Details)Mar 3 2016, 12:37 PM

@Harjotsingh: Have you checked T60032 for some underlying background? :)

Haritha28 claimed this task.Mar 4 2016, 5:10 PM

@Aklapper Yes , I have understood that we have to make a script that populates pp_sortkey column by taking values from pp_value.

What I would like to know -

  1. Wouldn't we require credentials like dbname and dbpass to connect to the database ?
  1. As the task is assigned, can I still work on this issue ?

@Haritha28: As you are assignee: Are you actively working on this? Any news / progress to share, or questions?

@Aklapper : As I am busy and I won't be able to work on it, I am putting the project up for grabs.

Haritha28 removed Haritha28 as the assignee of this task.Mar 20 2016, 7:51 AM
MtDu claimed this task.May 24 2017, 7:58 PM
MtDu added subscribers: Catrope, MtDu.

Working on this now, under guidance of @Catrope , should be done soon. :)

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptMay 24 2017, 7:58 PM

Change 355541 had a related patch set uploaded (by MtDu; owner: MtDu):
[mediawiki/core@master] Add a maintenance script for populating pp_sortkey

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

Change 355541 merged by jenkins-bot:
[mediawiki/core@master] Add a maintenance script for populating pp_sortkey

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

matej_suchanek updated the task description. (Show Details)
matej_suchanek removed a subscriber: wikibugs-l-list.