search

RomaGilyov / laravel-csv-importer

Flexible and reliable way to import, parse, validate and transform your csv files with laravel
MIT License
16 stars 9 forks source link

disable mutex logic #1

Open meyer59 opened 6 years ago

meyer59 commented 6 years ago

hi, Thanks first the great package, i am running into this error message: {"message":"Csv importer supports only: file, memcached and redis cache drivers"} I would like to know if there is any way to disable the mutex logic ? I don't need it at all, and my cache driver is database

RomaGilyov commented 6 years ago

Hi, Will make new option in configurations for this, "mutex" => true/false today, but if you have large files and calculations I wouldn't recommend to disable the mutex, will probably make mutex cache driver configurable as well in order to keep mutex work and keep your own cache system, probably something like this: "mutex_driver" => "redis"

meyer59 commented 6 years ago

that will be great! also if the driver could be database ?

RomaGilyov commented 6 years ago

Will be a bit more of work for this, don't have much time currently, maybe a bit later.

meyer59 commented 6 years ago

Please tag the new version so we can composer update, thanks.

RomaGilyov commented 6 years ago

I've made the configurable cache driver, you can composer require rgilyov/laravel-csv-importer dev-master now, everything should work there. Facing some minor issues with trevis configurations and test passing, so won't make new release version for now. During the week will make the tests pass and then will let you know that you can change "rgilyov/laravel-csv-importer": "dev-master" on "rgilyov/laravel-csv-importer": "^1.0" in your composer.json file.

meyer59 commented 6 years ago

Ok great thank you