vscode-sqltools icon indicating copy to clipboard operation
vscode-sqltools copied to clipboard

Published 20 hours ago • verified publisher icon mtxr

fix: fixes slow formatting of queries with large no. of lines

Open sivaramasubramanian opened this issue 1 year ago • 1 comments

Related to #977

This change removes RegEx-based trimming to improve performance. There may be other places where performance could be improved. After this change, the query mentioned in #977 gets formatted in under 1s.

Describe here what is this PR about and what we are achieving merging this.

Thank you for your contribution! Before submitting this PR, please make sure:

  • [x] Your code builds clean without any errors or warnings
  • [x] You have made the needed changes to the docs
  • [x] You have written a description of what is the purpose of this pull request above

sivaramasubramanian avatar Sep 17 '22 15:09 sivaramasubramanian

@mtxr do you see any downside to this change?

gjsjohnmurray avatar Sep 21 '22 12:09 gjsjohnmurray