Yang Zhang
Yang Zhang
> In the long term, I think we will only register spark sql functions after all the needed functions are supported. Currently, presto sql functions are registered just because the...
@rui-mo @PHILO-HE would you help review this PR first?
@mbasmanova would you help review this PR? thank you!
@mbasmanova would you help take a look? thank you!
> @Yohahaha Looks good, but CI is red. Please, take a look. thank you for the review, comments are fixed, but fuzzer job in CI seems broken due to lack...
> > fuzzer job in CI seems broken due to lack resources to execute, I check other merged PR also failed in fuzzer job > > @Yohahaha Thank you for...
hi @kgpai , I have fix conflict, would you help import again? thank you!
Hi, could you share the current status?
> > Hi, could you share the current status? > > @Yohahaha, this pr depends on a velox pr: [facebookincubator/velox#9425](https://github.com/facebookincubator/velox/pull/9425). I will try to push the review progress. why not...
> Hi @Yohahaha, it's strange that a few UT fails. Do you have any clues? sorry, I dont have enough time to find root cause and continue iterating this PR...