Skip to content
This repository was archived by the owner on Jan 28, 2021. It is now read-only.

*: implement subquery expressions #835

Merged
merged 1 commit into from
Oct 14, 2019

Conversation

erizocosmico
Copy link
Contributor

Closes #59

@erizocosmico erizocosmico requested a review from a team October 8, 2019 15:04
Copy link
Contributor

@agarciamontoro agarciamontoro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work. Reviewing such a big addition in go-mysql-server made me learn a lot! :)

There are some minor nitpicking comments below.

@erizocosmico erizocosmico force-pushed the feature/subquery-exprs branch from a887a9e to e78ff1b Compare October 9, 2019 07:14
@erizocosmico
Copy link
Contributor Author

Done

@erizocosmico erizocosmico requested a review from a team October 9, 2019 07:14
@erizocosmico erizocosmico force-pushed the feature/subquery-exprs branch from e78ff1b to b533f9f Compare October 9, 2019 07:15
Copy link
Contributor

@ajnavarro ajnavarro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@erizocosmico erizocosmico force-pushed the feature/subquery-exprs branch from b533f9f to 321f2a9 Compare October 9, 2019 09:49
@erizocosmico
Copy link
Contributor Author

Changed supported.md

@ajnavarro ajnavarro requested a review from a team October 10, 2019 10:07
Copy link
Contributor

@juanjux juanjux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing work!

@ajnavarro
Copy link
Contributor

@erizocosmico could you rebase to fix CI? thanks!

Signed-off-by: Miguel Molina <miguel@erizocosmi.co>
@erizocosmico
Copy link
Contributor Author

Done

@erizocosmico erizocosmico force-pushed the feature/subquery-exprs branch from 321f2a9 to 258a735 Compare October 11, 2019 09:51
@ajnavarro ajnavarro requested a review from carlosms October 11, 2019 10:05
@ajnavarro ajnavarro merged commit 151d56b into src-d:master Oct 14, 2019
@erizocosmico erizocosmico deleted the feature/subquery-exprs branch October 15, 2019 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support sqlparser.Subquery statement
4 participants