Log a message if file system caching is disabled on coordinator #21666
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
If a connector does not support caching on a Coordinator, Trino instantiates Alluxio JMX statistics but quietly disables caching. As a result, it appears that caching is instantiated and is operating. Consequently, it's really confusing when caching doesn't work but there is no indication why.
This PR will issue a warning on startup to let the user know that caching has been disabled for the Coordinator.
Considering this is a trivial PR, I didn't create a ticket. But if that is appropriate, I will open an issue.
Additional context and related issues
N/A
Release notes
(x) This is just a logging message, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: