You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "justaparth (via GitHub)" <gi...@apache.org> on 2023/05/09 17:14:22 UTC

[GitHub] [spark] justaparth commented on pull request #41108: [SPARK-43427] spark protobuf: modify deserialization behavior of unsigned integer types

justaparth commented on PR #41108:
URL: https://github.com/apache/spark/pull/41108#issuecomment-1540562071

   cc @rangadi i've made a draft implementation here but just wanted to get your thoughts quickly on:
   
   1. does the problem / solution make sense?
   2. should we make this behavior the default or should we add an option to turn it on? 
   
   thanks 🙏 


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org