Ajay George
Ajay George
@rschlussel / @lliiun-z Can we merge this, since this PR introduces `ThriftMetastoreUtil.LAST_DATA_COMMIT_TIME` which is used by https://github.com/facebookexternal/presto-facebook/pull/1907 which got merged today. The build is broken due to this.
@Chongliang-Utopia Please squash the commits and use a clean commit message.
@harveer07 Please sign the CLA as well.
@beinan There are compilation errors. Can you take a look. ``` Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:compile (default-compile) on project presto-cache: Compilation failure Error: /home/runner/work/presto/presto/presto-cache/src/main/java/com/facebook/presto/cache/alluxio/CacheValidatingInputStream.java:[106,47] cannot find symbol Error: symbol: variable...
@beinan There are a few test failures . Can you take a look. ``` Error: Failures: Error: TestAlluxioCachingFileSystem.testBasicReadWithAsyncRestoreFailure:261->checkMetrics:468 expected [0] but found [90] Error: TestAlluxioCachingFileSystem.testSyncRestoreFailure » Test Method TestAlluxioCachingFileSystem.testSyncRestoreFailure()[pri:0, instance:com.facebook.presto.cache.alluxio.TestAlluxioCachingFileSystem@2ddc8ecb]...
@branimir-vujicic This requires a release note since this is introducing a new feature. Can you please add that in.
@branimir-vujicic Are the Velox changes needed as a part of this PR. ? Seems unrelated. Can you please remove them.
Maven checks are failing. Cannot be merged in the present form.
Multiple tests are failing and the PR cannot be merged in the current form.
@rguillome Please squash the 6 commits in the PR (some of them are merge commits) Also refer to Release Notes wiki : https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines The current PR release notes does not...