Skip to content

Support selecting from table-returning functions with queryFunction#241

Merged
shane-circuithub merged 1 commit intomasterfrom
query-function
Jul 11, 2023
Merged

Support selecting from table-returning functions with queryFunction#241
shane-circuithub merged 1 commit intomasterfrom
query-function

Conversation

@shane-circuithub
Copy link
Copy Markdown
Contributor

Depends on opaleye#567.

@shane-circuithub
Copy link
Copy Markdown
Contributor Author

@ocharles I'm also tempted to change Rel8.Expr.Function.function to just (Table Expr input, Sql DBType output) => input -> Expr output instead of the weird variadic argument stuff we have currently. Then you could just supply multiple arguments with a tuple. I think that's better for type inference. What do you think?

@shane-circuithub shane-circuithub force-pushed the query-function branch 2 times, most recently from 1a7948c to 3401f76 Compare June 18, 2023 21:24
@shane-circuithub
Copy link
Copy Markdown
Contributor Author

This would fix #71.

@shane-circuithub shane-circuithub changed the base branch from master to function July 9, 2023 10:46
@shane-circuithub shane-circuithub force-pushed the function branch 3 times, most recently from 9962101 to fec9a3a Compare July 11, 2023 12:27
@shane-circuithub shane-circuithub force-pushed the query-function branch 2 times, most recently from f37849a to 851d700 Compare July 11, 2023 13:01
Base automatically changed from function to master July 11, 2023 13:05
@shane-circuithub shane-circuithub merged commit 9f372dc into master Jul 11, 2023
@shane-circuithub shane-circuithub deleted the query-function branch July 11, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant