-
-
Notifications
You must be signed in to change notification settings - Fork 194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
issues-179 Add support for view #293
base: master
Are you sure you want to change the base?
Conversation
I need more time for support backends, because SQLite, MySQL and Postgres are very different in the details. |
4337d07
to
160b7bd
Compare
@tyt2y3 @billy1624, hello. POC for support What to do? In |
@tyt2y3 @billy1624 any ideas? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @ikrivosheev, sorry for the delay. What a huge PR loll
Notice a few things. For, create view:
OR REPLACE
is supported by MySQL, MariaDB and PostgreSQL, SQLite wasn't supported- However,
IF NOT EXISTS
is supported by SQLite but not the other three db TEMPORARY
is supported by PostgreSQL and SQLite but not the other two dbRECURSIVE
is supported only by PostgreSQLWITH [ CASCADED | LOCAL ]
...
And, the list goes on. I think it's better to document these db specific features later in this PR. Or, even, dump some of the feature that supported by only one db.
References:
I understood this. My question was: #293 (comment) |
I think ViewCreateStatement should impl QueryStatementBuilder. |
status? |
@ian-p-cooke hello! I am working on support Postgres array in SeaQuery. After I will return back to this PR. |
72da2a0
to
ccc18fb
Compare
Well, It's time to continue working on this PR. |
Any news on this? |
Finishing View Support - POC
what is the current state of this PR? |
PR Info
Depends on:
Adds
Add support for: create, delete, rename
VIEW
.TODO: