prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
16.06k stars 5.38k forks source link

[Doc] Document all the currently supported SQL operation for postgresql connector #22245

Closed Akanksha-kedia closed 7 months ago

Akanksha-kedia commented 8 months ago

Document all the currently supported SQL operations for postgresql connector along with an example similar to https://prestodb.io/docs/current/connector/iceberg.html#sql-support

Expected Behavior or Use Case

To provide more concrete details to users. Once we have documentation for postgresql connector, we can document similar supported SQL operations for other JDBC connectors as well.

Presto Component, Service, or Connector

Presto Docs(postgresql)

Possible Implementation

Example Screenshots (if appropriate):

presently only :

Screenshot 2024-03-20 at 9 24 49 AM

Context

To provide more concrete details to users

Akanksha-kedia commented 8 months ago

we could have a section along with all the supported postgreSQL operations, and their supported example, and mention limitations for respective Operation if any similar to https://prestodb.io/docs/current/connector/iceberg.html#sql-support. I think consistency of doc structure helps users of the docs, so I think changing this page to a format more similar to that of the Iceberg docs and alos mysql is a good idea.

Akanksha-kedia commented 8 months ago

@steveburnett

Sreesruth commented 8 months ago

hii, @Akanksha-kedia , @joshk @electrum @jvanzyl @oggy i would like to work on this issue, thankyou.

Akanksha-kedia commented 8 months ago

Hello Thanks for looking into this.But i have already created prs you can review.

On Thu, 21 Mar 2024 at 6:37 PM, Sree @.***> wrote:

hii, @Akanksha-kedia https://github.com/Akanksha-kedia , @joshk https://github.com/joshk @electrum https://github.com/electrum @jvanzyl https://github.com/jvanzyl @oggy https://github.com/oggy i would like to work on this issue, thankyou.

— Reply to this email directly, view it on GitHub https://github.com/prestodb/presto/issues/22245#issuecomment-2012249112, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVL2AZWDDQLCNLXSDWJVTLDYZLLPXAVCNFSM6AAAAABE6XPBQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJSGI2DSMJRGI . You are receiving this because you were mentioned.Message ID: @.***>

steveburnett commented 8 months ago

Hello Thanks for looking into this.But i have already created prs you can review.

I believe the relevant PR for this issue is PR 22246.

Sreesruth commented 6 months ago

Hello Thanks for looking into this.But i have already created prs you can review. On Thu, 21 Mar 2024 at 6:37 PM, Sree @.> wrote: hii, @Akanksha-kedia https://github.com/Akanksha-kedia , @joshk https://github.com/joshk @electrum https://github.com/electrum @jvanzyl https://github.com/jvanzyl @oggy https://github.com/oggy i would like to work on this issue, thankyou. — Reply to this email directly, view it on GitHub <#22245 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVL2AZWDDQLCNLXSDWJVTLDYZLLPXAVCNFSM6AAAAABE6XPBQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJSGI2DSMJRGI . You are receiving this because you were mentioned.Message ID: @.>

okay, Thankyou.