[GitHub] [druid] vtlim commented on a diff in pull request #14169: NVL function docs update

["vtlim (via GitHub)" <gi...@apache.org>]

vtlim commented on code in PR #14169:
URL: https://github.com/apache/druid/pull/14169#discussion_r1179412351


##########
docs/querying/sql-scalar.md:
##########
@@ -270,6 +270,6 @@ The [DataSketches extension](../development/extensions-core/datasketches-extensi
 |`CASE WHEN boolean_expr1 THEN result1 \[ WHEN boolean_expr2 THEN result2 ... \] \[ ELSE resultN \] END`|Searched CASE.|
 |`NULLIF(value1, value2)`|Returns NULL if value1 and value2 match, else returns value1.|
 |`COALESCE(value1, value2, ...)`|Returns the first value that is neither NULL nor empty string.|
-|`NVL(expr, expr-for-null)`|Returns `expr-for-null` if `expr` is null (or empty string for string type).|
+|`NVL(value1, value2)`|Returns value1 if value1 is not null, otherwise value2.|

Review Comment:
   ```suggestion
   |`NVL(value1, value2)`|Returns `value1` if `value1` is not null, otherwise `value2`.|
   ```



-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org