Naveen Mahadevuni

Results 44 comments of Naveen Mahadevuni

24134 add session property orc use column names

> yes sure, I have a draft but I need some advice on where to put it. Can i add it to TestHiveIntegrationSmokeTest.java or should it go somewhere else? @davve14...

24134 add session property orc use column names

Just have one comment, looks good @davve14. After addressing the comment, please squash all your commits.

24134 add session property orc use column names

@hantangwangd @tdcmeehan Can you please review this?

Support reading Iceberg split with equality deletes

Thanks @czentgr. I have fixed the issue.

Add parquet bloom filter read support for int,bigint,string columns

@Yuhta Thanks for the review. I have addressed your comments. Please review.

Add parquet bloom filter read support for int,bigint,string columns

For the config/session property, I thought we would need a feature flag to disable/enable when wanted. The session property helps in testing.

Add parquet bloom filter read support for int,bigint,string columns

@Yuhta I have addressed your comments. Request to please review again.

Add parquet bloom filter read support for int,bigint,string columns

Reopening

Add parquet bloom filter read support for int,bigint,string columns

Ran some performance tests on Mac with file hive metastore and data in local storage. Results below. Data size is 1M rows in a single parquet file with 27 rowgroups....

Add parquet bloom filter read support for int,bigint,string columns

@Yuhta @yingsu00 Can you please review this?