phalcon-demo icon indicating copy to clipboard operation
phalcon-demo copied to clipboard

[NFR] Offer SQLite as the default database for the demo

Open AnnoyingTechnology opened this issue 6 years ago • 5 comments

Just a suggestion,

Use SQLite instead of MySQL as the backend for this demo. As this would lower the required dependencies and setup time for those who want to get a quick glance at the framework.

AnnoyingTechnology avatar Jul 02 '19 06:07 AnnoyingTechnology

@AnnoyingTechnology Good idea. Could you send a patch?

sergeyklay avatar Jul 03 '19 20:07 sergeyklay

I currently have no experience with Phalcon, soo... I'll see what I can do

AnnoyingTechnology avatar Jul 04 '19 07:07 AnnoyingTechnology

Will make this an option for the next minor release. Would like to keep support for MySQL as well since it's a common setup for a lot of users.

ruudboon avatar Jan 07 '20 12:01 ruudboon

it's a common setup for a lot of users.

Probably for high traffic use cases and in production. But for a demo it is quite useless. It requires more dependencies and configuration time, and doesn't make up for those in the context of a demo.

AnnoyingTechnology avatar Jan 07 '20 12:01 AnnoyingTechnology

It requires more dependencies and configuration time, and doesn't make up for those in the context of a demo.

That's why I keep both and you can decide in you env your setup.

ruudboon avatar Jan 07 '20 12:01 ruudboon