Allowed memory size exhausted

[Zubair-Iftikhar]

Note: php.ini memory_limit = 150M
Working fine with 99 record. But with 45K record memory issue.i update memory_limit 128 to 150 but same issue..remember i'm using Chunk Reading..

PHP Fatal error:  Allowed memory size of 157286400 bytes exhausted (tried to allocate 115749884 bytes) in E:\workspace\sites\dropshipping\vendor\phpoffice\phpspreadsheet\src\PhpSpreads
heet\Reader\Xlsx.php on line 303

   Symfony\Component\Debug\Exception\FatalErrorException  : Allowed memory size of 157286400 bytes exhausted (tried to allocate 115749884 bytes)

  at E:\workspace\sites\dropshipping\vendor\phpoffice\phpspreadsheet\src\PhpSpreadsheet\Reader\Xlsx.php:303
    299|         // Sadly, some 3rd party xlsx generators don't use consistent case for filenaming
    300|         //    so we need to load case-insensitively from the zip file
    301| 
    302|         // Apache POI fixes
  > 303|         $contents = $archive->getFromName($fileName, 0, ZipArchive::FL_NOCASE);
    304|         if ($contents === false) {
    305|             $contents = $archive->getFromName(substr($fileName, 1), 0, ZipArchive::FL_NOCASE);
    306|         }
    307| 


   Whoops\Exception\ErrorException  : Allowed memory size of 157286400 bytes exhausted (tried to allocate 115749884 bytes)

  at E:\workspace\sites\dropshipping\vendor\phpoffice\phpspreadsheet\src\PhpSpreadsheet\Reader\Xlsx.php:303
    299|         // Sadly, some 3rd party xlsx generators don't use consistent case for filenaming
    300|         //    so we need to load case-insensitively from the zip file
    301| 
    302|         // Apache POI fixes
  > 303|         $contents = $archive->getFromName($fileName, 0, ZipArchive::FL_NOCASE);
    304|         if ($contents === false) {
    305|             $contents = $archive->getFromName(substr($fileName, 1), 0, ZipArchive::FL_NOCASE);
    306|         }
    307| 

  Exception trace:

  1   Whoops\Run::handleError("Allowed memory size of 157286400 bytes exhausted (tried to allocate 115749884 bytes)", "E:\workspace\sites\dropshipping\vendor\phpoffice\phpspreadsheet\s
rc\PhpSpreadsheet\Reader\Xlsx.php")
      E:\workspace\sites\dropshipping\vendor\filp\whoops\src\Whoops\Run.php:454

  2   Whoops\Run::handleShutdown()
      [internal]:0

class ProductsImport implements ToCollection, WithChunkReading, WithHeadingRow, WithEvents
{
    use  Importable, RegistersEventListeners;

    protected $chunkSize = 10;
    protected $margin;

    /**
     * @param array $row
     *
     * @return \Illuminate\Database\Eloquent\Model|null
     */
    public function collection(Collection $products)
    {

        foreach ($products as $product) {
            if (Product::where('supplier_product_id', $product['id'])->first()) {
                continue;
            }
            $property_list = $product['property_list'] ?? null;
            $packaging_details = $product['packaging_details'] ?? null;
            $description = $product['description'] ?? null;
            $newProduct = Product::create([
                'supplier_product_id' => $product['id'] ?? null,
                'name' => $product['product_title'] ?? null,
                'description' => $description . $property_list . $packaging_details,
                'active' => 0,
                'status' => ProductStatusEnum::PENDING,
                'min_price' => $product['price'] ? $product['price'] + ($product['price'] * ($this->margin / 100)) : null,
                'extra_info' => [
                    'images' => $product['image_urls'] ?? null
                ],
            ]);
        }
    }

     public function chunkSize(): int
    {
        return $this->chunkSize;
    }

    public function setMargin($margin): void
    {
        $this->margin = $margin;
    }
}

[patrickbrouwers]

I would recommend using a bit more than 150mb, phpspreadsheet will always require memory to read the spreadsheet, even if using chunk reading. Also note that doing where's and create's use memory too. I would also recommend using bigger chunks and possibly using should queue so memory is reset after each chunk.