In T130947#2152392, @MaxSem wrote:

T130947.patch1 KBDownload

Looks good to me.

However, the error reporting for this patch isn't very good. Even something as simple as

'error' => function( $status ) { throw new FatalError( $status->getWikiText() ); } ]

for the error callback of PoolCounterDoWork would probably be ok if we don't expect errors to happen very often.

To that end, here's an alternative version:

In T130947#2200959, @Bawolff wrote:

To that end, here's an alternative version:

Use pool counter for gen diffs with better error handling2 KBDownload

LGTM, let's deploy it.

In T130947#2216252, @MaxSem wrote:

In T130947#2200959, @Bawolff wrote:

To that end, here's an alternative version:

Use pool counter for gen diffs with better error handling2 KBDownload

LGTM, let's deploy it.

Cool. Assuming things go smoothly with the DC transition, I'll probably deploy this tomorrow afternoon.