Page MenuHomeSealhub

Add compilation from sealious `Query` to SQL Prepared statement string
Closed, ResolvedPublic2 Points

Description

We are slowly adding support for SQL in sealious (Probably Postgresql is going to be first sql suported). Next step is to add new method in all Query classes that will compile "mongo objects" to sql strings. This strings are supposed to be Prepared statement. Tests already exist for this functionality and were added in this commit:

https://hub.sealcode.org/rSfc2e5d74e37f1232d4ed0611dd2b9ce50124d75e
https://node-postgres.com/features/pooling
https://www.commandprompt.com/education/how-the-prepare-statement-works-in-postgresql/

These tests will need some adjustments. At the moment it expects SQL string insteand of Prepared statement and uses toPipeline method. We should create new method as toPipeline works for mongo and this method should return object with Prepared statement string and array of parameters to pass.

Next tasks will appear in the future since we are designing this solution on the spot. We have also original task that is now obsolete but it can have some useful information for the future T2150

Details

Tested by
kuba-orlik
Coded by
FilipI
Reviewed by
kuba-orlik
Action required from
FilipI

Revisions and Commits

Event Timeline

FilipI merged a task: Unknown Object (Maniphest Task).Oct 31 2024, 18:09
FilipI added a subscriber: kuba-orlik.
jenkins-user updated Coded by, added: FilipI.
jenkins-user updated Action required from, added: FilipI.
FilipI updated the task description. (Show Details)
FilipI added a revision: Restricted Differential Revision.Oct 31 2024, 19:50
kuba-orlik set the point value for this task to 2.Nov 4 2024, 14:47
jenkins-user moved this task from Doing to Review on the Kanban board.
jenkins-user moved this task from Review to Doing on the Kanban board.
jenkins-user updated Reviewed by, added: kuba-orlik.
jenkins-user moved this task from Doing to Review on the Kanban board.
jenkins-user updated Action required from, added: kuba-orlik; removed: FilipI.
jenkins-user assigned this task to FilipI.
jenkins-user moved this task from Review to Ready for testing on the Kanban board.
jenkins-user updated Action required from, added: AmeliaB, Koossa, karolwojtowicz87, brtkuu, prostyfranek, zbartosz, radoptak; removed: kuba-orlik.
jenkins-user updated Tested by, added: kuba-orlik.
jenkins-user moved this task from Ready for testing to Ready to land on the Kanban board.
jenkins-user updated Action required from, added: FilipI; removed: AmeliaB, Koossa, karolwojtowicz87, brtkuu, prostyfranek, zbartosz, radoptak.