[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&focusedCommentId=15896066#comment-15896066 ] ASF GitHub Bot commented on FLINK-5955: --- Github user shaoxuan-wang commented on the

[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 enabl

[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&focusedCommentId=15896065#comment-15896065 ] ASF GitHub Bot commented on FLINK-5955: --- Github user shaoxuan-wang closed the pull r

[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 as

[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&focusedCommentId=15896063#comment-15896063 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on the

[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&focusedCommentId=15896059#comment-15896059 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 @@ abstr

[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&focusedCommentId=15896058#comment-15896058 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 Ag

[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&focusedCommentId=15896057#comment-15896057 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 Ag

[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 @@ abstra

[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&focusedCommentId=15896056#comment-15896056 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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&focusedCommentId=15896029#comment-15896029 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 @@ a

[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 @@ a

[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&focusedCommentId=15896000#comment-15896000 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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&focusedCommentId=15895999#comment-15895999 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 @@ abstra

[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 @@ import

[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&focusedCommentId=15895997#comment-15895997 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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&focusedCommentId=15895996#comment-15895996 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 Aggregat

[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&focusedCommentId=15895995#comment-15895995 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895993#comment-15895993 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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&focusedCommentId=15895987#comment-15895987 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 @@ abstr

[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&focusedCommentId=15895986#comment-15895986 ] ASF GitHub Bot commented on FLINK-5956: --- Github user shaoxuan-wang commented on a di

[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 Ag

[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&focusedCommentId=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 wel

[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&focusedCommentId=15895924#comment-15895924 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895925#comment-15895925 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in

[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 DataSetA

[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&focusedCommentId=15895906#comment-15895906 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895915#comment-15895915 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895914#comment-15895914 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895921#comment-15895921 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895917#comment-15895917 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895916#comment-15895916 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895918#comment-15895918 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895923#comment-15895923 ] ASF GitHub Bot commented on FLINK-5963: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895922#comment-15895922 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895912#comment-15895912 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895920#comment-15895920 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895919#comment-15895919 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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 Decima

[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&focusedCommentId=15895909#comment-15895909 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895911#comment-15895911 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895910#comment-15895910 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895908#comment-15895908 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895913#comment-15895913 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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&focusedCommentId=15895907#comment-15895907 ] ASF GitHub Bot commented on FLINK-5956: --- Github user fhueske commented on a diff in

[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 cla

[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 cla

[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 Deci

[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 java.

[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 Aggregat

[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 AggregateUt

[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 Deci

[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 cl

[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 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_r104297285 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/functions/aggfunctions/AvgAggFunction.scala --- @@ -290,6 +317,19 @@ class Decim

[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 Aggregat

[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 @@ abstrac

[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 cla

[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 @@ abstrac

[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 not

[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&focusedCommentId=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&focusedCommentId=15895808#comment-15895808 ] Steve Loughran commented on FLINK-5706: --- I should add that my current stance with us

[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&focusedCommentId=15895648#comment-15895648 ] Steve Loughran edited comment on FLINK-5706 at 3/4/17 6:24 PM: -

[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&focusedCommentId=15895769#comment-15895769 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a d

[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&focusedCommentId=15895770#comment-15895770 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a d

[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&focusedCommentId=15895771#comment-15895771 ] ASF GitHub Bot commented on FLINK-5963: --- Github user sunjincheng121 commented on a d

[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 D

[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&focusedCommentId=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 pro

[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&focusedCommentId=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` * `DropWizardMeterWrapper`

[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 Compo

[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 a

[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&focusedCommentId=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&focusedCommentId=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&focusedCommentId=15895723#comment-15895723 ] ASF GitHub Bot commented on FLINK-3679: --- Github user rmetzger commented on a diff in

[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 `Deser

[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&focusedCommentId=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&focusedCommentId=15895720#comment-15895720 ] Wei-Che Wei commented on FLINK-4816: Hi [~ram_krish] How about make the return value

  1   2   >