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

ASF GitHub Bot commented on FLINK-10259:
----------------------------------------

fhueske opened a new pull request #6641: [FLINK-10259] Fix key extraction for 
GroupWindows.
URL: https://github.com/apache/flink/pull/6641
 
 
   ## What is the purpose of the change
   
   Fixes the key validation for SQL queries with group window aggregates that 
are emitted to an `UpsertStreamTableSink`.
   
   ## Brief change log
   
   * Handle window properties (start, end, rowtime) in one equivalence group 
because all property attributes identify the same window.
   * Add tests for SQL `INSERT INTO` of different result types to different 
types of table sinks.
   
   ## Verifying this change
   
   * Run the added tests.
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): **no**
     - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: **no**
     - The serializers: **no**
     - The runtime per-record code paths (performance sensitive): **no**
     - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Yarn/Mesos, ZooKeeper: **no**
     - The S3 file system connector: **no**
   
   ## Documentation
   
     - Does this pull request introduce a new feature? **no**
     - If yes, how is the feature documented? **n/a**
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Key validation for GroupWindowAggregate is broken
> -------------------------------------------------
>
>                 Key: FLINK-10259
>                 URL: https://issues.apache.org/jira/browse/FLINK-10259
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Fabian Hueske
>            Assignee: Fabian Hueske
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.6.1, 1.7.0, 1.5.4
>
>
> WindowGroups have multiple equivalent keys (start, end) that should be 
> handled differently from other keys. The {{UpdatingPlanChecker}} uses 
> equivalence groups to identify equivalent keys but the keys of WindowGroups 
> are not correctly assigned to groups.
> This means that we cannot correctly extract keys from queries that use group 
> windows.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to