Skip to content

Do not wrap subqueries of combined queries, fixes #328. - #329

Open
juhovh wants to merge 1 commit into
korma:masterfrom
juhovh:union-fix
Open

Do not wrap subqueries of combined queries, fixes #328.#329
juhovh wants to merge 1 commit into
korma:masterfrom
juhovh:union-fix

Conversation

@juhovh

@juhovh juhovh commented Nov 1, 2015

Copy link
Copy Markdown
Contributor

More information in the related issue.

@juhovh

juhovh commented Nov 10, 2015

Copy link
Copy Markdown
Contributor Author

Any comments or suggestions on how to improve this in case the proposed fix is not ok?

@immoh

immoh commented Nov 10, 2015

Copy link
Copy Markdown
Member

Apologies for not responding earlier.

I don't like duplicating code, and I don't like the fallback if we don't know that it is needed.

I suppose it is possible to have (raw "select * from foo") as a subquery, but I'm fine with supporting only subselect there. One possible solution would be to extract subselect handling without wrapping to a separate function and call that function from both map-val and sql-combination-query.

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.

2 participants