Bumped precommit with sqlfluff version 4.1.0

[Sanket17052006]

Closes #11919

Changes:

• Bumped sqlfluff from 4.0.4 to 4.1.0 .
• Removed prod-partial-dump.sql from the exclusion list — it had fixable linting issues.
• Updated the exclusion comments to describe why the remaining two files are excluded.

1. patron_data.sql — contains raw tab-separated COPY data rows with \N nulls that sqlfluff cannot parse.
2. import-partial.sql — uses a psql variable placeholder (:source) inside a standard SQL COPY command that sqlfluff cannot parse

Testing

• pre-commit run --all-files
• docker compose run --rm home make test

Screenshot

N/A

Stakeholders

@RayBB

[RayBB]

@Sanket17052006 can you please open two new issues in the sqlfluff repo for the two features that are missing?
Just like someone did here: sqlfluff/sqlfluff#7512

Then link them in this PR.

[Sanket17052006]

I have opened the issues sqlfluff/sqlfluff#7696 and sqlfluff/sqlfluff#7697 for the missing features.

[RayBB]

Could you please create a new issue (similar to the one you're closing) mentioning the two new issues you created and saying we should update the pre-commit rule when those are merged.

Then in .pre-commit-config.yaml please mention that issue number in the comment so people know to go look at it when they wonder why we have these exceptions.

In any case, this is looking go so far and your work is appreciated! 👍

[Sanket17052006]

Thank you! @RayBB, I created issue #12245 and the mentioned the sqlfluff issues.
Also updated the .pre-commit-config.yaml comment to reference it.

[RayBB]

Locally pre-commit is working. Also these small changes to the query look very safe to me.

CC: @cdrini since he's the solr guru.

Thanks for your work following up on this one!