You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/02/26 14:45:49 UTC

[GitHub] viirya commented on issue #23797: [WIP][SPARK-26856][PYSPARK] Python support for from_avro and to_avro APIs

viirya commented on issue #23797: [WIP][SPARK-26856][PYSPARK] Python support for from_avro and to_avro APIs
URL: https://github.com/apache/spark/pull/23797#issuecomment-467465429
 
 
   > Hmm , @gengliangwang, @cloud-fan and @viirya, do you maybe have an idea about how we include Avro function APIs in Python side? I think it's reasonable to include Python API since we have it in Java/Scala too.
   
   The current approach looks okay to me too. The advantage of it is that it is consistent with existing way.
   
   Providing a separate Python file somewhere requires extra setup (like `py-files`) when users want to use the Python API.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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