Exasol: Add lua adapter scripts (7.1+)

[sti0]

Brief summary of the change made

Allow lua adapter scripts which are introduced in 7.1. Furthermore I removed the optional flag for the script language definition since this is mandatory in 7.1+.

Are there any other side effects of this change that we should be aware of?

May break scripts which not defined a language (<7.1). This could easy fixed by adding a script language parameter.

Pull Request checklist

• Please confirm you have completed any of the necessary steps below.

• Included test cases to demonstrate any code changes, which may be one or more of the following:

  • .yml rule test cases in test/fixtures/rules/std_rule_cases.
  • .sql/.yml parser test cases in test/fixtures/dialects (note YML files can be auto generated with tox -e generate-fixture-yml).
  • Full autofix test cases in test/fixtures/linter/autofix.
  • Other.

• Added appropriate documentation for the change.

• Created GitHub issues for any relevant followup/future enhancements if appropriate.

[tunetheweb]

Could we add a LUA test case to make sure it doesn’t regress in future?

Also no concerns of < 7.1 users not using that optional clause still? Or is it arguably best practice to include to anyway so happy to make non-optional?

[codecov]

Codecov Report

Merging #2801 (901aa24) into main (48d14ac) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #2801   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          163       163           
  Lines        12310     12309    -1     
=========================================
- Hits         12310     12309    -1     

Impacted Files	Coverage Δ
src/sqlfluff/dialects/dialect_exasol.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48d14ac...901aa24. Read the comment docs.

[sti0]

Also no concerns of < 7.1 users not using that optional clause still? Or is it arguably best practice to include to anyway so happy to make non-optional?

Like I said, it's easy to fix when adding a language keyword (this is still the prefered way). I think that adapter scripts are rarely used so this should not have great impact. Furthermore I think that we should add support for the latest version since 7.0.x will be EOL in mid 2022.

Could we add a LUA test case to make sure it doesn’t regress in future?

I don't have a working test case since I don't used this functionallity before. There is no example in the docs. I will add a dummy one to make sure the parsing works as expected.

[tunetheweb]

LGTM assuming all tests pass!