Remove Profiler specific code from fileOpPerfTest

[lhc/web/wiklou.git] / maintenance / fileOpPerfTest.php

diff --git a/maintenance/fileOpPerfTest.php b/maintenance/fileOpPerfTest.php
index 4cb5e10..4b6c619 100644 (file)
--- a/maintenance/fileOpPerfTest.php
+++ b/maintenance/fileOpPerfTest.php
@@ -21,11 +21,8 @@
  * @ingroup Maintenance
  */
 
-$initialTime = microtime( true );
-$wgProfiler = array( 'class' => 'ProfilerSimpleText' );
 error_reporting( E_ALL );
-
-require_once( __DIR__ . '/Maintenance.php' );
+require_once __DIR__ . '/Maintenance.php';
 
 /**
  * Maintenance script to test fileop performance.
@@ -35,7 +32,7 @@ require_once( __DIR__ . '/Maintenance.php' );
 class TestFileOpPerformance extends Maintenance {
        public function __construct() {
                parent::__construct();
-               $this->mDescription = "Test fileop performance";
+               $this->addDescription( 'Test fileop performance' );
                $this->addOption( 'b1', 'Backend 1', true, true );
                $this->addOption( 'b2', 'Backend 2', false, true );
                $this->addOption( 'srcdir', 'File source directory', true, true );
@@ -52,22 +49,17 @@ class TestFileOpPerformance extends Maintenance {
                        $backend = FileBackendGroup::singleton()->get( $this->getOption( 'b2' ) );
                        $this->doPerfTest( $backend );
                }
-
-               $profiler = Profiler::instance();
-               $profiler->setTemplated( true );
-
-               //NOTE: as of MW1.21, $profiler->logData() is called implicitly by doMaintenance.php.
        }
 
        protected function doPerfTest( FileBackend $backend ) {
-               $ops1 = array();
-               $ops2 = array();
-               $ops3 = array();
-               $ops4 = array();
-               $ops5 = array();
+               $ops1 = [];
+               $ops2 = [];
+               $ops3 = [];
+               $ops4 = [];
+               $ops5 = [];
 
                $baseDir = 'mwstore://' . $backend->getName() . '/testing-cont1';
-               $backend->prepare( array( 'dir' => $baseDir ) );
+               $backend->prepare( [ 'dir' => $baseDir ] );
 
                $dirname = $this->getOption( 'srcdir' );
                $dir = opendir( $dirname );
@@ -79,14 +71,14 @@ class TestFileOpPerformance extends Maintenance {
                        if ( $file[0] != '.' ) {
                                $this->output( "Using '$dirname/$file' in operations.\n" );
                                $dst = $baseDir . '/' . wfBaseName( $file );
-                               $ops1[] = array( 'op' => 'store',
-                                       'src' => "$dirname/$file", 'dst' => $dst, 'overwrite' => 1);
-                               $ops2[] = array( 'op' => 'copy',
-                                       'src' => "$dst", 'dst' => "$dst-1", 'overwrite' => 1 );
-                               $ops3[] = array( 'op' => 'move',
-                                       'src' => $dst, 'dst' => "$dst-2", 'overwrite' => 1 );
-                               $ops4[] = array( 'op' => 'delete', 'src' => "$dst-1" );
-                               $ops5[] = array( 'op' => 'delete', 'src' => "$dst-2" );
+                               $ops1[] = [ 'op' => 'store',
+                                       'src' => "$dirname/$file", 'dst' => $dst, 'overwrite' => 1 ];
+                               $ops2[] = [ 'op' => 'copy',
+                                       'src' => "$dst", 'dst' => "$dst-1", 'overwrite' => 1 ];
+                               $ops3[] = [ 'op' => 'move',
+                                       'src' => $dst, 'dst' => "$dst-2", 'overwrite' => 1 ];
+                               $ops4[] = [ 'op' => 'delete', 'src' => "$dst-1" ];
+                               $ops5[] = [ 'op' => 'delete', 'src' => "$dst-2" ];
                        }
                        if ( count( $ops1 ) >= $this->getOption( 'maxfiles', 20 ) ) {
                                break; // enough
@@ -97,7 +89,7 @@ class TestFileOpPerformance extends Maintenance {
 
                $method = $this->hasOption( 'quick' ) ? 'doQuickOperations' : 'doOperations';
 
-               $opts = array( 'force' => 1 );
+               $opts = [ 'force' => 1 ];
                if ( $this->hasOption( 'parallelize' ) ) {
                        $opts['parallelize'] = ( $this->getOption( 'parallelize' ) === 'true' );
                }
@@ -107,7 +99,7 @@ class TestFileOpPerformance extends Maintenance {
                $e = ( microtime( true ) - $start ) * 1000;
                if ( $status->getErrorsArray() ) {
                        print_r( $status->getErrorsArray() );
-                       exit(0);
+                       exit( 0 );
                }
                $this->output( $backend->getName() . ": Stored " . count( $ops1 ) . " files in $e ms.\n" );
 
@@ -116,7 +108,7 @@ class TestFileOpPerformance extends Maintenance {
                $e = ( microtime( true ) - $start ) * 1000;
                if ( $status->getErrorsArray() ) {
                        print_r( $status->getErrorsArray() );
-                       exit(0);
+                       exit( 0 );
                }
                $this->output( $backend->getName() . ": Copied " . count( $ops2 ) . " files in $e ms.\n" );
 
@@ -125,7 +117,7 @@ class TestFileOpPerformance extends Maintenance {
                $e = ( microtime( true ) - $start ) * 1000;
                if ( $status->getErrorsArray() ) {
                        print_r( $status->getErrorsArray() );
-                       exit(0);
+                       exit( 0 );
                }
                $this->output( $backend->getName() . ": Moved " . count( $ops3 ) . " files in $e ms.\n" );
 
@@ -134,7 +126,7 @@ class TestFileOpPerformance extends Maintenance {
                $e = ( microtime( true ) - $start ) * 1000;
                if ( $status->getErrorsArray() ) {
                        print_r( $status->getErrorsArray() );
-                       exit(0);
+                       exit( 0 );
                }
                $this->output( $backend->getName() . ": Deleted " . count( $ops4 ) . " files in $e ms.\n" );
 
@@ -143,11 +135,11 @@ class TestFileOpPerformance extends Maintenance {
                $e = ( microtime( true ) - $start ) * 1000;
                if ( $status->getErrorsArray() ) {
                        print_r( $status->getErrorsArray() );
-                       exit(0);
+                       exit( 0 );
                }
                $this->output( $backend->getName() . ": Deleted " . count( $ops5 ) . " files in $e ms.\n" );
        }
 }
 
 $maintClass = "TestFileOpPerformance";
-require_once( RUN_MAINTENANCE_IF_MAIN );
+require_once RUN_MAINTENANCE_IF_MAIN;