[jira] [Updated] (FLINK-5964) Change TypeSerializers to allow construction of immutable types

2017-03-04 Thread Jayson Minard (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jayson Minard updated FLINK-5964: - Affects Version/s: 1.1.4 > Change TypeSerializers to allow construction of immutable types >

[jira] [Commented] (FLINK-5955) Merging a list of buffered records will have problem when ObjectReuse is turned on

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896066#comment-15896066 ] ASF GitHub Bot commented on FLINK-5955: --- Github user shaoxuan-wang commented on the issue:

[GitHub] flink issue #3464: [FLINK-5955] [table] Merging a list of buffered records w...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on the issue: https://github.com/apache/flink/pull/3464 Sure @fhueske, this PR is closed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[jira] [Commented] (FLINK-5955) Merging a list of buffered records will have problem when ObjectReuse is turned on

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896065#comment-15896065 ] ASF GitHub Bot commented on FLINK-5955: --- Github user shaoxuan-wang closed the pull request at:

[GitHub] flink pull request #3464: [FLINK-5955] [table] Merging a list of buffered re...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang closed the pull request at: https://github.com/apache/flink/pull/3464 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] flink issue #3470: [FLINK-5956] [table] Add retract method for aggregateFunc...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on the issue: https://github.com/apache/flink/pull/3470 @fhueske Thanks for the review. I have addressed all your comments and updated the PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896063#comment-15896063 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on the issue:

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896059#comment-15896059 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104304129 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/AvgAggFunction.scala --- @@ -137,6 +146,15 @@

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896058#comment-15896058 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104304057 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896057#comment-15896057 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104303999 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104303912 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -49,13 +50,25 @@

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896056#comment-15896056 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[jira] [Updated] (FLINK-5855) Unprotected access to pendingFilesPerCheckpoint in BucketingSink

2017-03-04 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ted Yu updated FLINK-5855: -- Description: {code} handlePendingFilesForPreviousCheckpoints(restoredState.pendingFilesPerCheckpoint);

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896029#comment-15896029 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104302135 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/functions/aggfunctions/AggFunctionTestBase.scala --- @@ -36,14 +36,23 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301347 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/functions/aggfunctions/AggFunctionTestBase.scala --- @@ -36,14 +36,23 @@

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15896000#comment-15896000 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895999#comment-15895999 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301312 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -49,13 +50,25 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301172 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -23,10 +23,11 @@

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895997#comment-15895997 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895996#comment-15895996 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301147 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/MaxAggFunctionWithRetract.scala --- @@ -0,0 +1,205

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301102 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895995#comment-15895995 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895993#comment-15895993 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301092 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/MaxAggFunctionWithRetract.scala --- @@ -0,0 +1,205

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895987#comment-15895987 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104301022 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/AvgAggFunction.scala --- @@ -137,6 +146,15 @@

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895986#comment-15895986 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread shaoxuan-wang
Github user shaoxuan-wang commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104300989 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895927#comment-15895927 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on the issue:

[GitHub] flink issue #3472: [FLINK-5963] [table] Remove prepare mapper of DataSetAggr...

2017-03-04 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/3472 Hi @sunjincheng121, thanks for your review. I addressed you comments. Cheers, Fabian --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895924#comment-15895924 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895925#comment-15895925 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in the pull

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104298836 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetPreAggFunction.scala --- @@ -0,0 +1,87 @@ +/*

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104298833 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/dataset/DataSetAggregate.scala --- @@ -87,47 +89,67 @@ class

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895906#comment-15895906 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895915#comment-15895915 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895914#comment-15895914 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895921#comment-15895921 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895917#comment-15895917 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895916#comment-15895916 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895918#comment-15895918 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895923#comment-15895923 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895922#comment-15895922 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895912#comment-15895912 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895920#comment-15895920 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895919#comment-15895919 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895910#comment-15895910 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895908#comment-15895908 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895913#comment-15895913 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895907#comment-15895907 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104298822 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetAggFunction.scala --- @@ -0,0 +1,103 @@ +/* +

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298400 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -165,9 +190,9 @@ class

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895909#comment-15895909 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[jira] [Commented] (FLINK-5956) Add retract method into the aggregateFunction

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895911#comment-15895911 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in the pull

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298390 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -148,12 +161,24 @@ class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298146 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -23,10 +23,11 @@ import

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298193 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -49,13 +50,25 @@ abstract

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298179 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -49,13 +50,25 @@ abstract

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104297801 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/MaxAggFunctionWithRetract.scala --- @@ -0,0 +1,205 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104296481 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298418 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/AggregateUtil.scala --- @@ -556,7 +566,9 @@ object

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298377 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -148,12 +161,24 @@ class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104297207 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/AvgAggFunction.scala --- @@ -137,6 +146,15 @@ abstract

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298708 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/MaxAggFunctionWithRetract.scala --- @@ -0,0 +1,205 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298360 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -67,9 +80,9 @@ abstract

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298555 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/functions/aggfunctions/AggFunctionTestBase.scala --- @@ -36,14 +36,23 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104297285 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/AvgAggFunction.scala --- @@ -290,6 +317,19 @@ class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298605 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/AggregateFunction.scala --- @@ -35,6 +36,18 @@ abstract class

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298517 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/functions/aggfunctions/AggFunctionTestBase.scala --- @@ -36,14 +36,23 @@

[GitHub] flink pull request #3470: [FLINK-5956] [table] Add retract method for aggreg...

2017-03-04 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3470#discussion_r104298261 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/SumAggFunction.scala --- @@ -49,13 +50,25 @@ abstract

[GitHub] flink issue #3434: [FLINK-5909] [gelly] Interface for GraphAlgorithm results

2017-03-04 Thread vasia
Github user vasia commented on the issue: https://github.com/apache/flink/pull/3434 Hi @greghogan, thank you for the PR. I didn't spot anything that needs fixing, but I'm wondering what's the motivation to add these interfaces. I see how `toVerboseString()` is useful, but

[jira] [Commented] (FLINK-5909) Interface for GraphAlgorithm results

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895874#comment-15895874 ] ASF GitHub Bot commented on FLINK-5909: --- Github user vasia commented on the issue:

[jira] [Commented] (FLINK-5706) Implement Flink's own S3 filesystem

2017-03-04 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895808#comment-15895808 ] Steve Loughran commented on FLINK-5706: --- I should add that my current stance with using S3 as a

[jira] [Comment Edited] (FLINK-5706) Implement Flink's own S3 filesystem

2017-03-04 Thread Steve Loughran (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895648#comment-15895648 ] Steve Loughran edited comment on FLINK-5706 at 3/4/17 6:24 PM: --- Stefan, I

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895769#comment-15895769 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a diff in the pull

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895770#comment-15895770 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a diff in the pull

[jira] [Commented] (FLINK-5963) Remove preparation mapper of DataSetAggregate

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895771#comment-15895771 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a diff in the pull

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread sunjincheng121
Github user sunjincheng121 commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104290767 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/dataset/DataSetAggregate.scala --- @@ -87,47 +89,67 @@ class

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread sunjincheng121
Github user sunjincheng121 commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104290608 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetAggFunction.scala --- @@ -0,0 +1,103 @@ +/*

[GitHub] flink pull request #3472: [FLINK-5963] [table] Remove prepare mapper of Data...

2017-03-04 Thread sunjincheng121
Github user sunjincheng121 commented on a diff in the pull request: https://github.com/apache/flink/pull/3472#discussion_r104292001 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetPreAggFunction.scala --- @@ -0,0 +1,87 @@

[jira] [Commented] (FLINK-4286) Have Kafka examples that use the Kafka 0.9 connector

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4286?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895758#comment-15895758 ] ASF GitHub Bot commented on FLINK-4286: --- Github user mylog00 commented on the issue:

[GitHub] flink issue #3270: [FLINK-4286] Have Kafka examples that use the Kafka 0.9 c...

2017-03-04 Thread mylog00
Github user mylog00 commented on the issue: https://github.com/apache/flink/pull/3270 Hi @rmetzger, i delete example for Kafka v0.8 and left only for Kafka v0.9. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[jira] [Commented] (FLINK-5965) Typo on DropWizard wrappers

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895757#comment-15895757 ] ASF GitHub Bot commented on FLINK-5965: --- GitHub user csokol opened a pull request:

[GitHub] flink pull request #3475: [FLINK-5965] Typo on DropWizard wrappers

2017-03-04 Thread csokol
GitHub user csokol opened a pull request: https://github.com/apache/flink/pull/3475 [FLINK-5965] Typo on DropWizard wrappers This PR fixes two small typos in the metrics doc: * `DropWizardHistogramWrapper` should be `DropwizardHistogramWrapper` *

[jira] [Created] (FLINK-5965) Typo on DropWizard wrappers

2017-03-04 Thread Francisco Sokol (JIRA)
Francisco Sokol created FLINK-5965: -- Summary: Typo on DropWizard wrappers Key: FLINK-5965 URL: https://issues.apache.org/jira/browse/FLINK-5965 Project: Flink Issue Type: Bug

[GitHub] flink issue #3438: [FLINK-4460] Allow ProcessFunction on non-keyed streams

2017-03-04 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/3438 I think the discussion of timestamps and additional interfaces is orthogonal to this PR: `KeyedProcessOperator` is a renaming of the pre-existing `ProcessOperator` and the new `ProcessOperator` is

[jira] [Commented] (FLINK-4460) Side Outputs in Flink

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895742#comment-15895742 ] ASF GitHub Bot commented on FLINK-4460: --- Github user aljoscha commented on the issue:

[jira] [Commented] (FLINK-4460) Side Outputs in Flink

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895737#comment-15895737 ] ASF GitHub Bot commented on FLINK-4460: --- Github user rmetzger commented on the issue:

[GitHub] flink issue #3438: [FLINK-4460] Allow ProcessFunction on non-keyed streams

2017-03-04 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/3438 In addition to throwing an exception, we should also expose `element.hasTimestamp()` to offer our users a clean way of checking for timestamps. Lets see what @uce or other reviewers think about

[GitHub] flink issue #3314: [FLINK-3679] DeserializationSchema should handle zero or ...

2017-03-04 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/3314 The change looks good to merge in my opinion. @tzulitai can you also have a quick look? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (FLINK-3679) Allow Kafka consumer to skip corrupted messages

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895723#comment-15895723 ] ASF GitHub Bot commented on FLINK-3679: --- Github user rmetzger commented on a diff in the pull

[GitHub] flink pull request #3314: [FLINK-3679] DeserializationSchema should handle z...

2017-03-04 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/3314#discussion_r104227652 --- Diff: docs/dev/connectors/kafka.md --- @@ -146,6 +146,10 @@ The Flink Kafka Consumer needs to know how to turn the binary data in Kafka into

[jira] [Commented] (FLINK-3679) Allow Kafka consumer to skip corrupted messages

2017-03-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895722#comment-15895722 ] ASF GitHub Bot commented on FLINK-3679: --- Github user rmetzger commented on the issue:

[jira] [Commented] (FLINK-4816) Executions failed from "DEPLOYING" should retain restored checkpoint information

2017-03-04 Thread Wei-Che Wei (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15895720#comment-15895720 ] Wei-Che Wei commented on FLINK-4816: Hi [~ram_krish] How about make the return value in

  1   2   >