chore: Update docstrings in presto.py

[PApostol]

Update some docstrings in superset/db_engine_specs/presto.py to match the input arguments.

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• 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

[codecov]

Codecov Report

Merging #17623 (24912cc) into master (0544bee) will decrease coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #17623      +/-   ##
==========================================
- Coverage   68.55%   68.36%   -0.19%     
==========================================
  Files        1601     1601              
  Lines       65271    65271              
  Branches     6966     6966              
==========================================
- Hits        44745    44624     -121     
- Misses      18636    18757     +121     
  Partials     1890     1890              

Flag	Coverage Δ
hive	?
mysql	81.80% <ø> (ø)
postgres	81.81% <ø> (ø)
presto	?
python	81.89% <ø> (-0.41%)	⬇️
sqlite	81.49% <ø> (ø)

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

Impacted Files	Coverage Δ
superset/db_engine_specs/presto.py	83.50% <ø> (-6.89%)	⬇️
superset/db_engines/hive.py	0.00% <0.00%> (-85.19%)	⬇️
superset/db_engine_specs/hive.py	70.27% <0.00%> (-16.99%)	⬇️
superset/views/database/mixins.py	81.03% <0.00%> (-1.73%)	⬇️
superset/connectors/sqla/models.py	86.85% <0.00%> (-1.58%)	⬇️
superset/models/core.py	89.26% <0.00%> (-0.74%)	⬇️
superset/db_engine_specs/base.py	88.15% <0.00%> (-0.39%)	⬇️
superset/utils/core.py	89.64% <0.00%> (-0.12%)	⬇️

---

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 0544bee...24912cc. Read the comment docs.

[villebro]

LGTM, thanks for the cleanup!

[john-bodley]

@PApostol and @villebro beyond this change theres a cool PyLint extension docparams which ensures that params, returns, and exceptions are documented correctly.

We use it on a number of projects at Airbnb and it's a great way to ensuring that functions are well documented and that the documentation is up to date. Without doubt the introduction of Mypy has removed some need for doc-strings however because: i) (sadly) Mypy doesn't handle exception typing, and ii) variable names may not contain sufficient detail, I sense there is still merit in having doc-params.