bradleyboy/tuql

Add support for views

ad-si opened this issue · 3 comments

ad-si commented

Is there a fundamental problem preventing this?

bradleyboy commented

@ad-si I'm not that familiar with views, so I guess that's the fundamental problem 😄

My 5 minute read seems to show that Sequelize (which tuql uses under the hood) doesn't support SQLite views, so that may be a blocker.

ad-si commented

I'm not that familiar with views

Well, views are just queries which yield a new (read only) table. Reading from a table or a view works exactly in the same way, so it's really straight forward.

My 5 minute read seems to show that Sequelize (which tuql uses under the hood) doesn't support SQLite views, so that may be a blocker.

What the 😳. I guess that's why I never use ORMs. Completely useless abstraction. Since this project is explicitly for SQLite databases, maybe you shouldn't use it either? I feel like having direct access to all SQLite features would be handy…

  • 👍1
bradleyboy commented

Completely useless abstraction. Since this project is explicitly for SQLite databases, maybe you shouldn't use it either?

Not completely useless, it allowed for this library to be created quickly without writing a bunch of SQL plumbing myself 🤷‍♂

Regardless, views are not something I have needed for this, happy to look at a PR that brings support for them though.

  • 👍1