EtienneLamoureux / TQVaultAE

Extra bank space for Titan Quest Anniversary Edition
MIT License
279 stars 62 forks source link

persistence layer #439

Closed helio-clementino closed 2 years ago

helio-clementino commented 2 years ago

I imagine you have already thought about using a relational/document database instead of json files. Could you share your thoughts?

hguy commented 2 years ago

Hi @helio-clementino , Yes i though about using SQLite/Firebird. There's pros and cons about it.

Pros

Cons

Conclusion

It's nice feature and there is real benefits but it need a lot of tests to be useful and reliable and most users won't need it or notice it. At that point, it's more of a splinter in our engineer brain than a real user need. If the underlying layers of code base were shared with a wider project like a webapi and/or webapp, these modifications could be justified for technical/performance reasons. That's my thoughts about it.

helio-clementino commented 2 years ago

Thanks for replying. Indeed it can take a considerable effort to make such changes and they may not be noticeable by most of the users. Keep it up with such great work!