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.

Sign up for GitHub

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

Jump to bottom

Linter - Turn off looping behaviour for rules that don't need it #3035

Closed
3 tasks done
WittierDinosaur opened this issue Apr 7, 2022 · 0 comments
Closed
3 tasks done

Linter - Turn off looping behaviour for rules that don't need it #3035

WittierDinosaur opened this issue Apr 7, 2022 · 0 comments
Assignees
@barrywhart
Labels
core Issues relating to core design or architecture of SQLFluff enhancement New feature or request performance

Comments

@WittierDinosaur
Copy link
Contributor

Search before asking

  • I searched the issues and found no similar issues.

Description

Things like capitalisation policy, which make use of the parse tree, should never need to loop, and the parsed element types shouldn't be changed by the linter. Turning off looping behaviour would be massively benefitial for performance

Use case

Dialect

Are you willing to work on and submit a PR to address the issue?

  • Yes I am willing to submit a PR!

Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@barrywhart barrywhart

Labels
core Issues relating to core design or architecture of SQLFluff enhancement New feature or request performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@barrywhart @tunetheweb @WittierDinosaur