[BEAM-3610] [SQL] Fix HOP definition

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: dsl-sql
• Assignee: Anton Kedin
• Reporter: Anton Kedin
• Created: 02/Feb/18 18:31
• Updated: 24/Jul/20 20:13
• Resolved: 02/Feb/18 19:22

Description

Current HOP implementation has parameters in wrong order, according to Calcite doc. Sliding window size and frequency are swapped

 

According to Calcite docs the syntax is:

HOP(timestamp_field, frequency_interval, window_size)

We have

HOP(timestamp_field, window_size, frequency_interval)