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

Missing mysql 8 statements #336

Open
andrewchernow opened this issue Apr 29, 2021 · 9 comments
Open

Missing mysql 8 statements #336

andrewchernow opened this issue Apr 29, 2021 · 9 comments
Labels
enhancement help wanted kind/support A request for adding support

Comments

@andrewchernow
Copy link

Any plans to add the VALUES and WITH (CTE) statements? I am in need of detecting any statement that can produce results.

@williamdes
Copy link
Member

Hi,
Any implementation would be very welcome :)
We are open to contributions, any help on this repository would be awesome

@andrewchernow
Copy link
Author

I wish I could. Between all my projects and work, I don't have the time. It is curious though, that mysql 8 was released 3 years ago and these new statements are still not supported? Yet, you committed 9 days ago?

@williamdes
Copy link
Member

I wish I could. Between all my projects and work, I don't have the time. It is curious though, that mysql 8 was released 3 years ago and these new statements are still not supported? Yet, you committed 9 days ago?

I understand

On this repository there is some work to support new syntaxes, the code is improved but not much new features are written.
I did not write the code for this repository, and you can have a look but it is not always easy to understand how the parsing works this is for me the main reason I did not start writing features here
But this is for sure a topic and I hope to get this moving on in the next months

@andrewchernow
Copy link
Author

Really? I mean, it can'y be too hard. Let me look. Maybe I can patch it to support these new statements. I am swamped but I will look at it further.

@williamdes
Copy link
Member

Really? I mean, it can'y be too hard

It's not that hard but for me it's discouraging
My first task when I will switch on this repository will be to change the tests output files to something readable by humans so anyone can check the diffs between objects when adding features

In fact I do not feel enough confortable about the implementing and not breaking things part of it.
One very good thing is that this project has a very good code coverage and I wrote the missing tests a while ago

Conclusion: a better way to dump objects in tests and then new features

I hope this was not to much a messy explanation

@andrewchernow
Copy link
Author

Got it. Sorry, my comment may have come off insulting. That was not my intent.

@williamdes
Copy link
Member

Got it. Sorry, my comment may have come off insulting. That was not my intent.

No worries, you are just asking good questions :)

@williamdes
Copy link
Member

WITH will be part of 5.5.0
Implemented in: #334

@williamdes
Copy link
Member

This will help tests output and their changes to be understood: 9f16b24

@williamdes williamdes added the kind/support A request for adding support label Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement help wanted kind/support A request for adding support
Projects
None yet
Development

No branches or pull requests

2 participants