search

Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud
https://www.alluxio.io
Apache License 2.0
6.87k stars 2.94k forks source link

WIP Commit - untested. Test incoming. #18683

Open ArnavMohan opened 2 months ago

ArnavMohan commented 2 months ago

What changes are proposed in this pull request?

Addresses the misleading metrics exposed by trino. Tests that expose this are in progress and will be added to this branch.

Why are the changes needed?

See AC-2870. Behavior currently untested by internal code, but is confirmed to not break existing trino tests.

Does this PR introduce any user facing changes?

Resolved the behavior of the CLIENT_CACHE_PAGE_READ_CACHE_TIME_NS and CLIENT_CACHE_PAGE_READ_EXTERNAL_TIME_NS trackers to be per spec.

alluxio-bot commented 2 months ago

Thank you for your pull request. In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement (CLA). It's all electronic and will take just a few minutes. Please download CLA form here, sign, and e-mail back to cla@alluxio.org