Naveen Mahadevuni
Naveen Mahadevuni
@rui-mo Can you please review this?
Thanks @yingsu00 . I have addressed your comments. Please review.
> @tdcmeehan What approach would you recommend here? Do you already have something in mind or have you started working on it? If so, we're happy to close this PR...
Closing this as its addressed in https://github.com/prestodb/presto/pull/25594
@yingsu00 Please review.
@yingsu00 addressed the comments here. Please check.
Thanks @hantangwangd. The tests failed in this PR are in `TestPrestoNativeCteExecutionParquet`, not the failure I saw in my PR. I have seen this in one of my test runs too....
> @nmahadevuni As shown by my experiment, it is the newly added test class `TestPrestoNativeIcebergGeneralQueries` that affected other test cases. When creating iceberg query runner using parameter `addStorageFormatToPath=false`, it would...
@anandamideShakyan Please check the failed tests. You can run the same on your local machine and then if not related, rerun the tests in CI.
@aditi-pandit Can you please have a look?