kevinwilfong

It looks like there's a conflict with trunk, could you please rebase?

I have the internal tests all passing except for the issue I highlighted above. Also it looks like there are a couple unresolved comments still from others "add test for...

Addressed in https://github.com/facebookincubator/velox/pull/8526

thanks! I'll reach out to @mbasmanova

Spoke with Krishna and he found this PR breaks Presto Native, he sent details to Deepak offline, I'm going to remove the ready-to-merge tag until that's addressed

Removing the ready-to-merge tag as their are pending changes requested

Hi @pramodsatya it looks like we lost track of this change as it was never merged, could you rebase and update the PR?