Upgrade to Peggy 3 - Githubissues

nene / sql-parser-cst

Parses SQL into Concrete Syntax Tree (CST)

GNU General Public License v2.0

115 stars 7 forks source link

Closed nene closed 4 months ago

nene commented 4 months ago

Some fiddling was needed to keep the custom import-move plugin working. But not too much hassle.

Need to measure the performance. On first glance it feels slower. But not 100% sure.

nene commented 4 months ago

Indeed. On my local machine running yarn generate in Peggy3 branch is about 50% slower:

nene commented 4 months ago

Made two significant performance improvements:

Used skipTypeComputation: true in ts-pegjs to eliminate the main culprit that's causing slowdown in that plugin.
Created a small plugin which disables the reportInfiniteRecursion pass. Using it only in watch-mode. This ensures fast build times during development, while still getting a protection against problems reported by this rule from the CI build.