Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21986#discussion_r207808648
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
 ---
    @@ -123,7 +125,10 @@ trait HigherOrderFunction extends Expression {
       }
     }
     
    -trait ArrayBasedHigherOrderFunction extends HigherOrderFunction with 
ExpectsInputTypes {
    +/**
    + * Trait for functions having as input one argument and one function.
    + */
    +trait UnaryHigherOrderFunction extends HigherOrderFunction with 
ExpectsInputTypes {
    --- End diff --
    
    I called it `Unary` as it gets one input and one function. Honestly I can't 
think of a better name without becoming very verbose. if you have a better 
suggestion I am happy to follow it. I will add the `nullSafeEval`, thanks!


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to