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 2021/09/15 02:23:08 UTC

[GitHub] [spark] yutoacts commented on pull request #33988: [SPARK-36683][SQL] Add new built-in SQL functions: SEC and CSC

yutoacts commented on pull request #33988:
URL: https://github.com/apache/spark/pull/33988#issuecomment-919646406


   > 
   > 
   > For the Python code, it's better to follow @HyukjinKwon 's comments in #33992
   
   OK, and seems like [#33992](https://github.com/apache/spark/pull/33992) could conflict with this PR. I'll wait for it to be merged before pushing test_functions.py.


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