trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From DaveBirdsall <...@git.apache.org>
Subject [GitHub] trafodion pull request #1486: [TRAFODION-3001] Add an Overview of *Datetime ...
Date Wed, 21 Mar 2018 20:20:16 GMT
Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1486#discussion_r176223257
  
    --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc
---
    @@ -150,67 +150,109 @@ See the individual entry for the function.
     == Datetime Functions
     
     These functions use either a datetime value expression as an argument or
    -return a result of datetime data type:
    +return a result of DATETIME, VALUE or CHAR:
    --- End diff --
    
    The old text was correct. It categorized datetime functions as being those that either
accept datetimes or produce them. The new change, though, would allow SUBSTRING as a Datetime
function (since it produces a CHAR output).


---

Mime
View raw message