Commit Graph

10 Commits (fbe76e37f3e86b5f598db73e7226bdc8b64305c8)

Author SHA1 Message Date
Christian Winther 2de9c70a0c
fix: don't restore memory limit after cities import
Since this command can only be invoked by CLI, the process will exit after a successful import, so restoring the transient PHP memory limit doesn't really have any affect.

In PHP 8.4, this throws the following error (which doesn't happen in 8.3 and below)

> [entrypoint / 11-first-time-setup.sh] - (stderr) 128769/128769 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%[2025-01-20 11:29:23] production.ERROR: Failed to set memory limit to 134217728 bytes (Current memory usage is 134746112 bytes) {"exception":"[object] (ErrorException(code: 0): Failed to set memory limit to 134217728 bytes (Current memory usage is 134746112 bytes) at /var/www/app/Console/Commands/ImportCities.php:140)

It seems to be a 8.4 behavior change, so removing the logic would make it go away
3 months ago
Dan Church c685e3e9f4
Fix global memory limit set in constructor
Instead, set it when running.

Having the set in the constructor was causing memory_limit to be changed
for "artisan queue:work" processes.
4 years ago
Daniel Supernault 91e58591be
Update ImportCities command, bump memory limit 6 years ago
Daniel Supernault d391deaf8a
Update ImportCities command, clean up 6 years ago
Daniel Supernault ea023fcf17
Update ImportCities command, check for corrupt or invalid checksum 6 years ago
Daniel Supernault 450602cbeb
Update Import Cities command, shorten country names 6 years ago
monkeyless 32a6100fed Add last progressbar advance 6 years ago
monkeyless 6d771819f4 Add checks back 6 years ago
monkeyless 27952883b6 Isert cities in chunks to increase performance 6 years ago
Daniel Supernault ae4af1675d
Add new commmand 6 years ago