-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add subquery test code #82
base: master
Are you sure you want to change the base?
Add subquery test code #82
Conversation
Hello @bourbonkk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly there :) but we still have an issue with loading query (with subquery) from json
@bourbonkk @jbguerraz - Any updates on this PR? |
@saketbairoliya2 No, it doesn't. |
@bourbonkk - So are we planning to merge this one or what's the further plan? |
This PR has only reviews left. |
Okay - May be @jbguerraz would be a better person to approve this PR. Waiting for him to get back. Thanks @bourbonkk |
Hi, I wrote a test code for subquery. #15 #78
@jbguerraz @saketbairoliya2 @vigith
There were two code modifications.
This is necessary to make a subquery.
To express this example code,