flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-6817) Fix NPE when preceding is not set in OVER window
Date Wed, 14 Jun 2017 08:37:00 GMT

    [ https://issues.apache.org/jira/browse/FLINK-6817?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16048869#comment-16048869

ASF GitHub Bot commented on FLINK-6817:

Github user fhueske commented on the issue:

    This should be the same code as #4055 because the windows should not have changed on the
`master` branch since the release right?
    If that's the case, I'm +1 for merging as well.
    btw. we usually do not open separate PRs for different branches (unless there is backporting
required due to changes on `master` branch). Instead, you can cherrypick the commit from the
`master` branch into the `release-1.3` branch, run the tests (on Travis), and commit to the
release branch.

> Fix NPE when preceding is not set in OVER window
> ------------------------------------------------
>                 Key: FLINK-6817
>                 URL: https://issues.apache.org/jira/browse/FLINK-6817
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Jark Wu
>            Assignee: Jark Wu
>            Priority: Minor
>             Fix For: 1.4.0
> When preceding is not set in over window , a NPE will be thrown:
> {code}
> val result = table
>       .window(Over orderBy 'rowtime as 'w)
>       .select('c, 'a.count over 'w)
> {code}
> {code}
> java.lang.NullPointerException
> 	at org.apache.flink.table.api.OverWindowWithOrderBy.as(windows.scala:97)
> {code}
> Preceding must be set in OVER window, so should throw a more explicit exception not a

This message was sent by Atlassian JIRA

View raw message