trino
trino copied to clipboard
trinodb
Align flush_metadata_cache procedure named args with other procedures
Description
Follow up to @colebow PR https://github.com/trinodb/trino/pull/13398
syncing flush_metadata_cache named args with other procedures.
Related issues, pull requests, and links
- PR https://github.com/trinodb/trino/pull/13398
Documentation
( ) No documentation is needed. (x) Sufficient documentation is included in this PR. ( ) Documentation PR is available with #prnumber. ( ) Documentation issue #issuenumber is filed, and can be handled later.
Release notes
( ) No release notes entries required. (x) Release notes entries required with the following suggested text:
# Hive
* Align `flush_metadata_cache` named args to other procedures
Do we / should we have any way to warn the user in case that the parameter names used are deprecated?
Do we / should we have any way to warn the user in case that the parameter names used are deprecated?
No haven't figured out any. We might put it in logs, but thats kinda not clear 🤷♂️ If there is some way it could be shown in CLI as a warning would be supper cool.
@kokosing all test passed only single Product Test failed. CI Suite-2 - but looks not related at all.
CI Suite-2 - but looks not related at all.
Please check the failure and report a flaky issue on Github. We cannot just accept flaky failures, we need to fix them eventually. Tracking them is important to understand which of those should be escalated.
Details
@kokosing done It's added to exiting flaky test report here https://github.com/trinodb/trino/issues/6799#issuecomment-1263713925