Improve performance of SqlParser::getNextPlaceholder() method

[Tigrov]

Q	A
Is bugfix?	❌
New feature?	❌
Breaks BC?	❌
Fixed issues

Related PR

• Improve performance of AbstractSqlParser class methods db#1159

[samdark]

Any benchmarks?

[Copilot]

Pull request overview

This PR optimizes PostgreSQL SQL parsing by speeding up how the next placeholder is found, specifically by making identifier skipping more efficient inside SqlParser::getNextPlaceholder().

Changes:

• Introduced a cached identifier character set (IDENTIFIER_CHARS) for fast scanning.
• Replaced the per-character match loop in skipIdentifier() with a strspn()-based implementation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.68%. Comparing base (da724f8) to head (c77e5f6).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #482      +/-   ##
============================================
- Coverage     96.70%   96.68%   -0.03%     
+ Complexity      408      406       -2     
============================================
  Files            67       67              
  Lines          1214     1206       -8     
============================================
- Hits           1174     1166       -8     
  Misses           40       40              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Tigrov]

Any benchmarks?

Yes, tested locally. Almost 2 times faster 🚀