docs: Add documentation on PostgreSQL extension #4124
Conversation
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Thanks!
Happy to merge.
One note — it's a shame that cargo pgrx doesn't make it easier to install, and requires cloning the repo. I would have hoped that cargo pgrx install could take a crates.io package.
It might be worth considering a one-command install option which would do many of these steps. It could probably be written in a build.rs such that cargo install plprql does much of it. There's always the curl ....sh | bash option otherwise...
Or maybe the convention is to use the postgres package repositories and investing there would be better?
| ``` | ||
| select prql('from rounds | filter match_id == 1, 'rounds_cursor'); | ||
| ``` |
There was a problem hiding this comment.
If you're up for it, a fuller example of how to pull records from the cursor would be great!
There was a problem hiding this comment.
Yes, will add
There was a problem hiding this comment.
Expanded on the examples and added tests for the example code
Co-authored-by: eitsupi <50911393+eitsupi@users.noreply.github.com>
for more information, see https://pre-commit.ci
That's an great idea. Will do that after I get .deb files packaged up. For now, I don't think we should block the PR on this. |
There was a problem hiding this comment.
Great summary! I think this is a really good balance between some preview without forking the whole plprql readme...
| 1 | 1001 | 1 | Player1 | 4 | 1 | ||
| 3 | 1001 | 2 | Player1 | 1 | 7 | ||
| (2 rows) | ||
| ``` |
There was a problem hiding this comment.
Great example!
This is very cool indeed, I really like this way of doing it...
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: eitsupi <50911393+eitsupi@users.noreply.github.com> (cherry picked from commit cd95bc6)
This PR adds documentation on the PL/PRQL PostgreSQL extension to
See also discussion in #725