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

    https://github.com/apache/spark/pull/15513#discussion_r84573952
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
 ---
    @@ -361,13 +465,27 @@ case class Factorial(child: Expression) extends 
UnaryExpression with ImplicitCas
     }
     
     @ExpressionDescription(
    -  usage = "_FUNC_(x) - Returns the natural logarithm of x with base e.",
    -  extended = "> SELECT _FUNC_(1);\n 0.0")
    +  usage = "_FUNC_(expr) - Returns the natural logarithm of x with base 
expr.",
    --- End diff --
    
    This should be "natural logarithm of expr" only. The base is e, but that's 
implied by natural logarithm. OK, you could say "natural logarithm (base e) of 
expr"


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to