aokolnychyi commented on pull request #31083:
URL: https://github.com/apache/spark/pull/31083#issuecomment-761077046


   Alright, I've updated this PR and I think there are NO major outstanding 
questions related to this particular PR.
   
   Minor open points to be addressed in this PR:
   - Usage of `V2Expression` alias (discussed 
[here](https://github.com/apache/spark/pull/31083/files#r555614899)).
   
   Other open points to be addressed after this PR:
   - Removing unnecessary repartitions (discussed 
[here](https://github.com/apache/spark/pull/31083/files#r553252442))
   - Ability to define the number of partitions (discussed 
[here](https://github.com/apache/spark/pull/31083/files#r555318573))
   - Support for Structured Streaming (discussed 
[here](https://github.com/apache/spark/pull/31083/files#r555393381))
   
   In order to make progress on this feature, I'd ask everybody to review this 
PR as I think it is close to go in. We can discuss other points in the 
follow-up PRs.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to