Jimmy Lu

Results 24 issues of Jimmy Lu

We are recently using xsimd to make our [Velox](https://github.com/facebookincubator/velox) query evaluation engine portable. One of the gap we found is xsimd does not support gather for different sizes of types...

We are recently using xsimd to make our [Velox](https://github.com/facebookincubator/velox) query evaluation engine portable. One of the gap we found is xsimd only supports constant permutation mask in `swizzle`. Can we...

Summary: Currently when we copy `ARRAY(VARCHAR)`, for each array offset, we call `acquireSharedStringBuffers` and this is relatively expensive. Similar inefficiency is also happening when we call `copy` repeatedly while copying...

CLA Signed
fb-exported

Summary: Before this change, intermediate aggregation node for `approx_percentile` in worker only sees a type signature of `VARBINARY -> VARBINARY`, thus not be able to figure out the actual value...

fb-exported

Summary: Regular anti join is different from null-aware anti join in that: 1. NULL in build side does not affect the result. They won't be joined and won't participate in...

CLA Signed
fb-exported

Summary: Currently in intermediate aggregation node, the function only receive the input type (intermediate type) and output type (intermediate type). Some functions (e.g. `avg(decimal)`) need to merge the accumulators differently...

CLA Signed
fb-exported

In the issue related to https://github.com/prestodb/presto/pull/18207, aggregate function signature is ignored during the matching and only name is matched, which results in some hard-to-understand error message. We should double check...

bug
function
aggregates

When building (`make debug`) on Apple M1, I run into these errors: ``` /Library/Developer/CommandLineTools/usr/bin/make cmake BUILD_DIR=debug BUILD_TYPE=Debug mkdir -p _build/debug && \ cmake -B \ "_build/debug" \ -DTREAT_WARNINGS_AS_ERRORS=1 -DENABLE_ALL_WARNINGS=1 -DVELOX_BUILD_MINIMAL="OFF"...

build

This is the change needed on presto_cpp side to implement the new exchange protocol. Implement `ExchangeSource::getDataSizes` and use it when the header `X-Presto-Get-Data-Size` exists (will be removed in the future...