You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by walterddr <gi...@git.apache.org> on 2018/06/23 15:02:27 UTC

[GitHub] flink pull request #6188: [FLINK-6846][Table API] add timestampAdd tableApi

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

    https://github.com/apache/flink/pull/6188#discussion_r197613157
  
    --- Diff: docs/dev/table/tableApi.md ---
    @@ -2804,6 +2804,17 @@ dateFormat(TIMESTAMP, STRING)
           </td>
         </tr>
     
    +    <tr>
    +      <td>
    +        {% highlight java %}
    +timestampAdd(unit, interval, timestamp)
    +{% endhighlight %}
    +      </td>
    +      <td>
    +        <p>Adds a (signed) integer interval to a timestamp. The unit for the interval is given by the unit argument, which should be one of the following values: <code>"SECOND"</code>, <code>"MINUTE"</code>, <code>"HOUR"</code>, <code>"DAY"</code>, <code>"WEEK"</code>, <code>"MONTH"</code>, <code>"QUARTER"</code>, or <code>"YEAR"</code>. E.g. <code>timestampAdd("WEEK", 1, '2003-01-02'.toDate)</code> leads to <code>2003-01-09</code>.</p>
    --- End diff --
    
    Let's make it consistent with `sql.md`, I believe there's no `"` for the agument values


---