eejbyfeldt commented on PR #41876:
URL: https://github.com/apache/spark/pull/41876#issuecomment-1658296113

   > Rebased to fix conflict with master branch.
   > 
   > @hvanhovell Would you be able to have a second look or know who is a 
suitable reviewer for this change?
   
   Seems like the spark-connect specs are broken after 
https://github.com/apache/spark/pull/40611 since the arrow serialization you 
implemented there does not support value classes. That seems like a limitation 
that should be addressed, but does not really seem like really related to the 
issue handled here.
   
   Did you consider value classes when implementing 
https://github.com/apache/spark/pull/40611? 


-- 
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.

To unsubscribe, e-mail: [email protected]

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