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

ability to send multiple resultsets from handleQuery #524

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

GoWebProd
Copy link

For example, if query in HandleQuery equal to:

SELECT 0 UNION SELECT 1;
SELECT 1 AS col1, 2 AS col2, 3 AS col3 UNION SELECT 4, 5, 6;

we need to return two result sets, but current realisation not allows to make it

@skoef
Copy link
Contributor

skoef commented Jun 21, 2021

I would love to see more of this PR. While I don't get how this got approved in the first place (some parts are completely irrelevant to this PR and the new function ChainResultSet is not even used), but I implemented something similar myself as well, which I like to either convert to something that could be merged upstream or could convert to whatever this PR could lead to.

Especially reading the multiple results from the wire into a chained resultset is something I would like to wrap my head around!

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.

3 participants