fix: SQL Statement on QUERY_LOGGER prints none to log

[cccs-rc]

SUMMARY

Attempting to fix issue described here: #11356

TEST PLAN

As per issue linked above:

1. Set QUERY_LOGGER to a function that will log the query (e.g. def log_sql_statements(database, query, schema, user, client, security_manager, log_params): logger.info("query is:" + str(query)) QUERY_LOGGER = log_sql_statements
2. Run any query from sql_lab
3. Check the logs

ADDITIONAL INFORMATION

• Has associated issue: SQL Statement on QUERY_LOGGER prints none to log #11356
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[villebro]

LGTM - thanks so much for the fix!

[codecov]

Codecov Report

Merging #14358 (d1aa81e) into master (a548b69) will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #14358      +/-   ##
==========================================
+ Coverage   79.78%   79.92%   +0.14%     
==========================================
  Files         943      944       +1     
  Lines       47748    47999     +251     
  Branches     6026     6026              
==========================================
+ Hits        38094    38363     +269     
+ Misses       9533     9515      -18     
  Partials      121      121              

Flag	Coverage Δ
hive	80.77% <100.00%> (+0.29%)	⬆️
mysql	81.03% <100.00%> (+0.28%)	⬆️
postgres	81.06% <100.00%> (+0.28%)	⬆️
python	81.44% <100.00%> (+0.25%)	⬆️
sqlite	80.67% <100.00%> (+0.28%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset/sql_lab.py	81.56% <100.00%> (ø)
superset/db_engine_specs/__init__.py	65.90% <0.00%> (-34.10%)	⬇️
superset/db_engine_specs/hive.py	87.60% <0.00%> (-3.24%)	⬇️
superset/datasets/commands/importers/v1/utils.py	57.14% <0.00%> (-1.77%)	⬇️
...erset/dashboards/commands/importers/v1/__init__.py	98.50% <0.00%> (-1.50%)	⬇️
superset/dashboards/dao.py	95.45% <0.00%> (-0.52%)	⬇️
superset/models/core.py	88.85% <0.00%> (-0.25%)	⬇️
superset/viz.py	55.49% <0.00%> (-0.16%)	⬇️
superset/reports/commands/execute.py	92.88% <0.00%> (-0.08%)	⬇️
superset/views/core.py	75.65% <0.00%> (-0.06%)	⬇️
... and 40 more

---

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 a548b69...d1aa81e. Read the comment docs.