dépôts
/
lhc
/
web
/
wiklou.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Yet another attempt to fix the populateIpChanges script
[lhc/web/wiklou.git]
/
maintenance
/
populateIpChanges.php
diff --git
a/maintenance/populateIpChanges.php
b/maintenance/populateIpChanges.php
index
ffb8c43
..
f7bcc12
100644
(file)
--- a/
maintenance/populateIpChanges.php
+++ b/
maintenance/populateIpChanges.php
@@
-46,6
+46,12
@@
then be available when querying for IP ranges at Special:Contributions.
TEXT
);
$this->addOption( 'rev-id', 'The rev_id to start copying from. Default: 0', false, true );
TEXT
);
$this->addOption( 'rev-id', 'The rev_id to start copying from. Default: 0', false, true );
+ $this->addOption(
+ 'max-rev-id',
+ 'The rev_id to stop at. Default: result of MAX(rev_id)',
+ false,
+ true
+ );
$this->addOption(
'throttle',
'Wait this many milliseconds after copying each batch of revisions. Default: 0',
$this->addOption(
'throttle',
'Wait this many milliseconds after copying each batch of revisions. Default: 0',
@@
-57,34
+63,41
@@
TEXT
public function doDBUpdates() {
$lbFactory = MediaWikiServices::getInstance()->getDBLoadBalancerFactory();
public function doDBUpdates() {
$lbFactory = MediaWikiServices::getInstance()->getDBLoadBalancerFactory();
+ $dbr = $this->getDB( DB_REPLICA, [ 'vslow' ] );
$dbw = $this->getDB( DB_MASTER );
$throttle = intval( $this->getOption( 'throttle', 0 ) );
$dbw = $this->getDB( DB_MASTER );
$throttle = intval( $this->getOption( 'throttle', 0 ) );
+ $maxRevId = intval( $this->getOption( 'max-rev-id', 0 ) );
$start = $this->getOption( 'rev-id', 0 );
$start = $this->getOption( 'rev-id', 0 );
- $end = $dbw->selectField( 'revision', 'MAX(rev_id)', false, __METHOD__ );
+ $end = $maxRevId > 0
+ ? $maxRevId
+ : $dbw->selectField( 'revision', 'MAX(rev_id)', false, __METHOD__ );
$blockStart = $start;
$revCount = 0;
$this->output( "Copying IP revisions to ip_changes, from rev_id $start to rev_id $end\n" );
while ( $blockStart <= $end ) {
$blockStart = $start;
$revCount = 0;
$this->output( "Copying IP revisions to ip_changes, from rev_id $start to rev_id $end\n" );
while ( $blockStart <= $end ) {
- $rows = $dbw->select(
+ $blockEnd = min( $blockStart + $this->mBatchSize, $end );
+ $rows = $dbr->select(
'revision',
[ 'rev_id', 'rev_timestamp', 'rev_user_text' ],
'revision',
[ 'rev_id', 'rev_timestamp', 'rev_user_text' ],
- [ "rev_id >= $blockStart", 'rev_user' => 0 ],
- __METHOD__,
- [ 'ORDER BY' => 'rev_id ASC', 'LIMIT' => $this->mBatchSize ]
+ [ "rev_id BETWEEN $blockStart AND $blockEnd", 'rev_user' => 0 ],
+ __METHOD__
);
);
- if ( !$rows || $rows->numRows() === 0 ) {
- break;
+ $numRows = $rows->numRows();
+
+ if ( !$rows || $numRows === 0 ) {
+ $blockStart = $blockEnd + 1;
+ continue;
}
}
- $this->output( "...checking $
this->mBatchSize
revisions for IP edits that need copying, " .
- "
starting with rev_id $blockStart
\n" );
+ $this->output( "...checking $
numRows
revisions for IP edits that need copying, " .
+ "
between rev_ids $blockStart and $blockEnd
\n" );
$insertRows = [];
foreach ( $rows as $row ) {
$insertRows = [];
foreach ( $rows as $row ) {
- //
Double-check to make sure this is
an IP, e.g. not maintenance user or imported revision.
+ //
Make sure this is really
an IP, e.g. not maintenance user or imported revision.
if ( IP::isValid( $row->rev_user_text ) ) {
$insertRows[] = [
'ipc_rev_id' => $row->rev_id,
if ( IP::isValid( $row->rev_user_text ) ) {
$insertRows[] = [
'ipc_rev_id' => $row->rev_id,
@@
-94,12
+107,8
@@
TEXT
$revCount++;
}
$revCount++;
}
-
- $blockStart = (int)$row->rev_id;
}
}
- $blockStart++;
-
$dbw->insert(
'ip_changes',
$insertRows,
$dbw->insert(
'ip_changes',
$insertRows,
@@
-109,6
+118,8
@@
TEXT
$lbFactory->waitForReplication();
usleep( $throttle * 1000 );
$lbFactory->waitForReplication();
usleep( $throttle * 1000 );
+
+ $blockStart = $blockEnd + 1;
}
$this->output( "$revCount IP revisions copied.\n" );
}
$this->output( "$revCount IP revisions copied.\n" );