You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Aihua Xu (JIRA)" <ji...@apache.org> on 2016/12/22 17:39:58 UTC

[jira] [Commented] (HIVE-15498) sum() over (order by c) should default the windowing spec to RangeBoundarySpec

    [ https://issues.apache.org/jira/browse/HIVE-15498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15770625#comment-15770625 ] 

Aihua Xu commented on HIVE-15498:
---------------------------------

patch-1: change the default to RangeBoundarySpec for the case sum() over (order by c)  to be consistent.

> sum() over (order by c) should default the windowing spec to RangeBoundarySpec
> ------------------------------------------------------------------------------
>
>                 Key: HIVE-15498
>                 URL: https://issues.apache.org/jira/browse/HIVE-15498
>             Project: Hive
>          Issue Type: Bug
>          Components: PTF-Windowing
>    Affects Versions: 2.1.0
>            Reporter: Aihua Xu
>            Assignee: Aihua Xu
>         Attachments: HIVE-15498.1.patch
>
>
> Currently {{sum() over (partition by a)}} without order by will default windowing to RangeBoundarySpec while  {{sum() over (partition by a order by c)}} will default to ValueBoundarySpec.
> From the comment 
> {noformat}
>   /*
>    * - A Window Frame that has only the /start/boundary, then it is interpreted as:
>          BETWEEN <start boundary> AND CURRENT ROW
>    * - A Window Specification with an Order Specification and no Window
>    *   Frame is interpreted as:
>          ROW BETWEEN UNBOUNDED PRECEDING AND CURRENT ROW
>    * - A Window Specification with no Order and no Window Frame is interpreted as:
>          ROW BETWEEN UNBOUNDED PRECEDING AND UNBOUNDED FOLLOWING
>    */
> {noformat}
> We were trying to set as "row between", not "range between". 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)