Add limit option for MSSQL query runner

[dvandonkelaar]

What type of PR is this?

• Refactor
• Feature
• Bug Fix
• New Query Runner (Data Source)
• New Alert Destination
• Other

Description

The limit query adds LIMIT 1000 to the end of the query, but this isn't working for MSSQL queries.
This change adds the possibility to add the query runner specified limit text after the SELECT statement, based on the limit_after_select variable.

How is this tested?

• Unit tests (pytest, jest)
• E2E Tests (Cypress)
• Manually
• N/A

Related Tickets & Documents

This issue is also discussed in #5773. No other issues

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

N/A

[justinclift]

Thanks for this @dvandonkelaar. It looks like this needs some formatting tweaks in order to pass the CI testing though.

Are you ok to get that fixed?

[dvandonkelaar]

Thanks for the quick reply! My apologies, apparently I ignored the non-installed linter errors.
Commint 7e63a0f fixes the errors.

[codecov]

Codecov Report

Attention: Patch coverage is 62.50000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 63.41%. Comparing base (2fe0326) to head (9f26303).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6704      +/-   ##
==========================================
- Coverage   63.42%   63.41%   -0.01%     
==========================================
  Files         162      162              
  Lines       13173    13186      +13     
  Branches     1819     1822       +3     
==========================================
+ Hits         8355     8362       +7     
- Misses       4522     4527       +5     
- Partials      296      297       +1     

Files	Coverage Δ
redash/query_runner/mssql.py	34.56% <100.00%> (+2.51%)	⬆️
redash/query_runner/mssql_odbc.py	36.14% <100.00%> (+2.39%)	⬆️
redash/query_runner/__init__.py	76.63% <40.00%> (-1.39%)	⬇️

[justinclift]

Looks good to me, and we can ignore the minor codecov problem in this instance. 😄

[justinclift]

@dvandonkelaar Thanks for getting this done, it fixes an important bug with the MSSQL driver. 😄