-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat: parse postgres trigram operators #1186
base: main
Are you sure you want to change the base?
Conversation
564cc9b
to
a957e87
Compare
It is missing tests for now. |
7f41050
to
290c704
Compare
290c704
to
93f0344
Compare
Let me know if you need any help with it! |
It seems to not be enough, so if you can spot what I'm missing would be great :) |
21def56
to
974ab0c
Compare
Thank you :) |
f49f679
to
409b6de
Compare
Sorry I've been a little slow to get to this change. I think it's great and thanks for the contribution. The thing I am slightly worried about is that unlike sqlfluff we shouldn't make extensions part of the default dialect. I think it should add the concept of modifiers, I'll have a look at doing this next. |
See https://www.postgresql.org/docs/current/pgtrgm.html
Fixes #1180