Yang Zhang

Results 16 comments of 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? > > @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...