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/03/04 02:54:10 UTC

[GitHub] [spark] HyukjinKwon edited a comment on issue #23882: [SPARK-26979][PySpark] Add missing column name support for some SQL functions

HyukjinKwon edited a comment on issue #23882: [SPARK-26979][PySpark] Add missing column name support for some SQL functions
URL: https://github.com/apache/spark/pull/23882#issuecomment-469099769
 
 
   Can we send what we talked here into the mailing list so that other committers and other contributors can refer when they review?
   
   If I am not mistaken, we're going to:
   
   1. Don't add string overriden ones anymore in Scala side
   2. Support string as columns in Python and R side since the codes are short
   3. Postpone to decide if we're going to drop string as column or not later in Scala side.
   
   Am I correct? Also, please leave reasons for each item in the email. _Please_ don't leave controversial reasons like Python's duck typing reason. IMHO It's orthogonal - semantic similarity isn't duck-typing.

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