Sergey Pershin

Properly process Final Selection in Reduce().

2

Summary: In Reduce() we ignored the fact that Final Selection might have been false before we set it to false ourselves. In that case we need to use different SelectivityVector....

Also check for Presto Native operator names in QueryStats.

5

Presto Native operator names are different. ## Description To properly report 'shuffled' and 'raw input' query stats when running Presto Native, we also check Presto Native operator names. ``` ==...

## Description Add 'control_query_utf8' and 'test_query_utf8' columns to the verifier query table. ## Motivation and Context We realized that existing 'control_query' and 'test_query' columns are of latin1 format, which results...

Allow Presto Coordinator to ignore (not throw) negative runtime metrics.

1

We recently had this situation: Coordinator's log was full of exception call stacks: java.lang.IllegalArgumentException: size is negative at io.airlift.units.Preconditions.checkArgument(Preconditions.java:26) at io.airlift.units.DataSize.(DataSize.java:58) at io.airlift.units.DataSize.succinctDataSize(DataSize.java:48) at io.airlift.units.DataSize.succinctBytes(DataSize.java:43) ... at com.facebook.presto.execution.scheduler.SqlQueryScheduler.getBasicStageStats(SqlQueryScheduler.java:853) ... at...

json_extract_scalar(), json_extract() and json_size() returns NULL in some cases when it could return a value.

2

Consider the 3 queries below, which all should have the same result: ``` SELECT json_extract_scalar(c0, '$["hostname"]') from (values ('{"failure_reason": None, "hostname": "twshared38330.04.ftw6.facebook.com"}')) t(c0); C++: twshared38330.04.ftw6.facebook.com Java: NULL SELECT json_extract_scalar(c0, '$["hostname"]')...