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/05/31 18:10:12 UTC

[GitHub] [spark] nvander1 opened a new pull request #24761: [SPARK-27905] [SQL] Add higher order function 'forall'

nvander1 opened a new pull request #24761: [SPARK-27905] [SQL] Add higher order function 'forall'
URL: https://github.com/apache/spark/pull/24761
 
 
   ## What changes were proposed in this pull request?
   
   Add's the higher order function `forall`, which tests an array to see if a predicate holds for every element.
   The function is implemented in `org.apache.spark.sql.catalyst.expressions.ArrayForAll`.
   The function is added to the function registry under the pretty name `forall`.
   
   ## How was this patch tested?
   
   I've added appropriate unit tests for the new ArrayForAll expression in
   `sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/HigherOrderFunctionsSuite.scala`.
   
   Also added tests for the function in `sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala`.
   
   Not sure who is best to ask about this PR so:
    @HyukjinKwon @rxin @gatorsmile @ueshin @srowen @hvanhovell @gatorsmile 

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