Github user xueyumusic commented on the issue:

    https://github.com/apache/flink/pull/6188
  
    @twalthr I looked around and realized that current `+` expression could 
support TimeInterval (TimePoint) addition, thus the timestampAdd api is 
duplicated and seems have no necessary. 
    
    so I only made these changes:
    1. add `1.week` expression
    2. modify `quarter` as `1.quarter` 
    3. modify Extract to support extracting QUARTER and WEEK, as suggested and 
verified by @walterddr 
    
    Please have a review, @twalthr @fhueske @walterddr @hequn8128  thanks for 
all of you,


---

Reply via email to