Thejdeep Gudivada

Results 8 comments of Thejdeep Gudivada

@zhouyejoe @otterc Updated the PR with the latest changes, please take a look. Thank you!

@otterc @zhouyejoe Fixed the failing tests, should be good for another review now. Thanks!

@steven-aerts Thanks for working on this feature. +1 to this PR. The lack of complex union type write support causes us problems too. Right now, since the standard Dataframe/Dataset APIs...

Thanks for reviewing @zhouyejoe @mridulm, addressed feedback.

Addressed comments, please take another look @otterc @mridulm @zhouyejoe Thanks!

@otterc @mridulm The test failure is from a rebased commit that added protobuf serialization changes. I had to rebase locally and force push, this rewrote commit history.

@mridulm `StageDataWrapper` was also added to protobuf serialization in #39192 (merged a few days ago) causing this build to fail. I have updated this PR to make more changes to...

Fixed failing tests and updated commit messages to reflect the overall changes. PTAL. Thanks