Naveen Mahadevuni

Address review comments for constraints optimization

1

Comments at https://github.com/prestodb/presto/pull/17917#pullrequestreview-1033413711 addressed ``` == NO RELEASE NOTE == ```

Make logical property classes immutable

1

Test plan - This PR only made the classes immutable. Existing tests work. ``` == NO RELEASE NOTE == ```

Fix the problem where buildSelectedType() makes the input type invalid

16

`buildSelectedType()` takes a constant typeWithId type tree and applies the selector on it to return a selected new type tree. However, it modifies the input typeWithId unexpectedly because it duplicates...

[native] Add positional deletes to Iceberg native tpcds e2e tests

2

## Description Added delete phase to the Iceberg tpc-ds native e2e tests. ## Motivation and Context Goal to add full data maintenance phases described in the tpc-ds spec to the...

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

4

This PR adds support to read a column's bloom filter if it exists and tests it to discard rowgroups that don't satisfy the filter. For e.g., if the query has...

Allow different namespaces in builtin function namespace manager

4

## Description This change allows functions registered via `Plugin->getFunctions()` SPI to use a different namespace. ## Motivation and Context At present, functions registered via getFunctions() SPI are registered in `"presto.default"...

Theta sketch output doesn't match between Java and CPP

5

I have run a simple theta sketch computation as below in Java and CPP ``` Union union = Union.builder().buildUnion(); union.update(1); union.update(2); CompactSketch compactSketch = union.getResult(); byte[] bytes = union.getResult().toByteArray() ```...

refactor(native): Fail on empty or non-existent config environment variable

1

## Description This change is to report error when config properties reader finds an empty or non-existent envrionment variable. ## Motivation and Context If environment variable is empty or non-existent,...

[native]: Implement Sketch Theta aggregate and scalar functions.

1

Need to implement Sketch Theta aggregate and scalar functions required for the new Iceberg statistics for number of distinct values computation introduced in Presto Java. ## Expected Behavior or Use...

fix(s3): Add retry logic for some errors from client side

6

Fixes https://github.com/facebookincubator/velox/issues/13061 This PR adds 1) Retry logic for client errors which are non-retriable by S3 SDK with Http Status code -1 and INTERNAL FAILURE and UNKNOWN FAILURE. 2) Adds...