Jimmy Lu

Results 156 comments of Jimmy Lu

@karteekmurthys So there is also an option for us to do the same thing as java to just sum in partial & intermediate aggregation. We still need to handle the...

@karteekmurthys It might not be easy in Velox. Since you have to handle overflow in final aggregation anyway, it might worth just use the same implementation in both and give...

@karteekmurthys The errors I saw are trivial ones so far, no worry for now

Hi @yingsu00 can you resolve the conflicts here? Otherwise this is ready to be merged

Generic signature is not enough, we need to pass resolved raw input types. A discussion is started with this: #2869

You need to build velox and presto_cpp as 2 separate .so if you want to enable parquet support. I remember @majetideepak has made it work, @majetideepak would you provide some...

@rui-mo I am going to work on this.

Hi @rui-mo , with #2906 both regular and null-aware anti join will be supported in Velox. Feel free to download the PR and try it even before it merges.