Manish

Results 73 comments of Manish

@alexjo2144 thanks for the review! please check, the refactored code.

@alexjo2144 would you mind checking it, and seeing if changes are looking ok now? Also, fixed a doc formatting and [TestIcebergMetastoreAccessOperations.java](https://github.com/trinodb/trino/pull/11206/files#diff-5467855d6faa4e78a4c95c2f64ac7f534bbf93f04d0f3fbc448129eab2c195a7) as well. thanks!

@findinpath @mosabua thanks for the review! will try to add the above changes today/tomorrow.

> > Spark already supports the iceberg's $data_files and $all_data_files metadata tables. Trino is already supporting $files. > > Spark has actually `$files` and `$all_data_files`. I would argue that (for...

@electrum request to please provide your feedback, thanks! meanwhile will review the maven-check failures.

> @electrum request to please provide your feedback, thanks! > > meanwhile will review the maven-check failures. fixed the maven failures as well, it was due to the doc formatting.

> @osscm did you sign the CLA? yes @findepi and @martint has updated it.

@phd3 thanks for helping in review! That will help to merge it soon!