[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added

2011-07-23 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r86179.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86179#c20056
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

Deploy plan:
*Merge and sync r87606 (removes some FORCE INDEX statements related to PK)
*Run path-fr_page_rev-index.sql but WITHOUT the "DROP COLUMN" statements 
(includes PK change)
*Copy populateRevTimestamp.php to cluster and run it, recoding the ending 
fr_rev_id (say LAST_REV)
*SVN update FR to 1.18 but don't scap
*Set $wgReadOnly = "Performing brief site maintenance, this should only take a 
few minutes." for FR wikis
*Run populateRevTimestamp.php --startrev=LAST_REV (this should be quick)
*Disable $wgReadOnly for FR wikis
*Scap

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added

2011-07-23 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r86179.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86179#c20057
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

Actually $wgReadOnly should be disabled *after* the scap.

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added

2011-07-24 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r86179.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86179#c20099
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

See http://wikitech.wikimedia.org/view/User:Aaron/flaggedrevs_timestamp_column

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added

2011-04-18 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r86179.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86179#c16077
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

r86192

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added

2011-05-01 Thread MediaWiki Mail
User "Aaron Schulz" posted a comment on MediaWiki.r86179.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/86179#c16474
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

Code changes still needed to make use of this. That will come soon.

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r86179]: New comment added, and revision status changed

2011-04-18 Thread MediaWiki Mail
User "Bryan" changed the status of MediaWiki.r86179.

Old Status: new
New Status: fixme

User "Bryan" also posted a comment on MediaWiki.r86179.

Full URL: 
https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/86179#c16075
Commit summary:

Overdue changes to `flaggedrevs` table:
* Removed unused ft_text and fr_comment fields
* Changed the PK to just rev_id (with an index on old PK fields)
* Added fr_rev_timestamp field with indexes
* Added a script to populate fr_rev_timestamp

Comment:

svn:eol-style on the .sql files.

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview