Skip to content
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

sql: remaining PostgREST work #91331

Closed
1 of 2 tasks
dikshant opened this issue Nov 5, 2022 · 3 comments
Closed
1 of 2 tasks

sql: remaining PostgREST work #91331

dikshant opened this issue Nov 5, 2022 · 3 comments
Labels
C-wishlist A wishlist feature. meta-issue Contains a list of several other issues. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@dikshant
Copy link

dikshant commented Nov 5, 2022

Now that we have initial support for PostgREST #69010 we need to figure out why UDFs aren't working as intended.

It might be also worth our effort to run PostgREST tests against CockroachDB to identify missing functionality.
https://github.com/PostgREST/postgrest/tree/main/test

PostgREST also relies on

Jira issue: CRDB-21203

@dikshant dikshant added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) meta-issue Contains a list of several other issues. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) T-sql-schema-deprecated Use T-sql-foundations instead labels Nov 5, 2022
@postamar postamar added C-wishlist A wishlist feature. and removed C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) labels Nov 10, 2022
@rafiss
Copy link
Collaborator

rafiss commented Nov 22, 2022

@dikshant would you be able to add more details here to expand on "UDFs aren't working as intended"? Which functionality is broken, and are there instructions for how we can reproduce the problems?

@exalate-issue-sync exalate-issue-sync bot removed the T-sql-schema-deprecated Use T-sql-foundations instead label Jan 4, 2023
@mgartner
Copy link
Collaborator

Should we close this until we have details about specific things we need to fix? @dikshant

@dikshant
Copy link
Author

Ah I meant to update this a while back and I forgot.

Yeah lets close this for now. I will reopen once I add the udf failure logs from postgrest in here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-wishlist A wishlist feature. meta-issue Contains a list of several other issues. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

No branches or pull requests

4 participants