do not optimize after commit

#36

src/Listener/JobEventSubscriber.php

@@ -186,10 +186,9 @@ public function postFlush(PostFlushEventArgs $eventArgs)
             $client->deleteByIds($this->entityToDocumentFilter->getDocumentIds($job));
         }
 
-        // commit to index & optimize it
+        // commit to index
         $client = $this->getSolrClient();
         $client->commit(true, false);
-        $client->optimize(1, true, false);
 
         // clear arrays. There could be multiple flushs per requests and we do not want
         // to process already processed entities.

test/SolrTest/Listener/JobEventSubscriberTest.php

@@ -94,19 +94,19 @@ protected function setUp(): void
             ->getMock();
         $options->method('getJobsPath')
             ->willReturn('/some/path');
-        
+
         $this->client = $this->getMockBuilder(\SolrClient::class)
             ->disableOriginalConstructor()
             ->getMock();
-        
+
         $this->manager = $this->getMockBuilder(Manager::class)
             ->disableOriginalConstructor()
             ->getMock();
         $this->manager->method('getClient')
             ->willReturn($this->client);
         $this->manager->method('getOptions')
             ->willReturn($options);
-        
+
         $this->entityToDocumentFilter = $this->getMockBuilder(EntityToDocumentFilter::class)
             ->disableOriginalConstructor()
             ->getMock();
@@ -137,17 +137,17 @@ public function testPrePersistShouldNotProcessNonJobDocument()
             ->getMock();
         $job->expects($this->never())
             ->method('isActive');
-        
+
         $event = $this->getMockBuilder(PreUpdateEventArgs::class)
             ->disableOriginalConstructor()
             ->getMock();
         $event->expects($this->once())
             ->method('getDocument')
             ->willReturn($job);
-        
+
         $this->subscriber->prePersist($event);
     }
-    
+
     /**
      * @param string $status
      * @param bool $shouldBeAdded
@@ -187,7 +187,7 @@ public function testPreUpdateShouldNotProcessNonJobDocument()
             ->method('getDocument');
         $event->expects($this->never())
             ->method('hasChangedField');
-        
+
         $this->subscriber->preUpdate($event);
     }
 
@@ -241,7 +241,7 @@ public function testPreUpdateWithChangedStatus($status, $shouldBeAdded, $shouldB
 
         $subscriber = $this->subscriber;
         $subscriber->preUpdate($event);
-        
+
         if ($shouldBeAdded) {
             $this->assertContains($job, $subscriber->getAdd());
         } else {
@@ -254,7 +254,7 @@ public function testPreUpdateWithChangedStatus($status, $shouldBeAdded, $shouldB
             $this->assertNotContains($job, $subscriber->getDelete());
         }
     }
-    
+
     /**
      * @covers ::postFlush()
      */
@@ -266,14 +266,14 @@ public function testPostFlushWithNoJobsToProcess()
             ->getMock();
         $subscriber->expects($this->never())
             ->method('getSolrClient');
-        
+
         $event = $this->getMockBuilder(PostFlushEventArgs::class)
             ->disableOriginalConstructor()
             ->getMock();
-        
+
         $subscriber->postFlush($event);
     }
-    
+
     /**
      * @param string $status
      * @param bool $shouldBeAdded
@@ -296,47 +296,45 @@ public function testPostFlushWithJobsToProcess($status, $shouldBeAdded, $shouldB
             ->method('hasChangedField')
             ->with($this->equalTo('status'))
             ->willReturn(true);
-        
+
         $this->subscriber->preUpdate($event);
-        
+
         if ($shouldBeAdded) {
             $document = new SolrInputDocument();
             $this->entityToDocumentFilter->expects($this->once())
                 ->method('filter')
                 ->with($this->identicalTo($job))
                 ->willReturn($document);
-            
+
             $this->client->expects($this->once())
                 ->method('addDocument')
                 ->with($this->identicalTo($document));
         }
-        
+
         if ($shouldBeDeleted) {
             $ids = [1, 2, 3];
             $this->entityToDocumentFilter->expects($this->once())
                 ->method('getDocumentIds')
                 ->with($this->identicalTo($job))
                 ->willReturn($ids);
-            
+
             $this->client->expects($this->once())
                 ->method('deleteByIds')
                 ->with($this->identicalTo($ids));
         }
-        
+
         if ($shouldBeAdded || $shouldBeDeleted) {
             $this->client->expects($this->once())
                 ->method('commit');
-            $this->client->expects($this->once())
-                ->method('optimize');
         }
-        
+
         $event = $this->getMockBuilder(PostFlushEventArgs::class)
             ->disableOriginalConstructor()
             ->getMock();
-        
+
         $this->subscriber->postFlush($event);
     }
-    
+
     /**
      * @covers ::factory()
      */
@@ -350,10 +348,10 @@ public function testFactory()
             ->withConsecutive([$this->equalTo('Solr/Options/Module')],
                               [$this->equalTo('Solr/Manager')])
             ->will($this->onConsecutiveCalls($options,$this->manager));
-        
+
         $this->assertInstanceOf(JobEventSubscriber::class, JobEventSubscriber::factory($container));
     }
-    
+
     /**
      * @return array
      */