[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82132367 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/stream/table/GroupWindowITCase.scala --- @@ -0,0 +1,777 @@ +/* + *

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551224#comment-15551224 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551226#comment-15551226 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551211#comment-15551211 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82131749 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/stream/table/GroupWindowITCase.scala --- @@ -0,0 +1,777 @@ +/* + *

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82135948 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -118,9 +109,9 @@ public void

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551284#comment-15551284 ] ASF GitHub Bot commented on FLINK-4348: --- Github user KurtYoung commented on a diff in the pull

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551341#comment-15551341 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82141352 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/operators.scala --- @@ -55,28 +56,27 @@ case class

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551169#comment-15551169 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82129916 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/scala/table/expressionDsl.scala --- @@ -325,33 +337,42 @@ trait

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82132349 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/stream/table/GroupWindowITCase.scala --- @@ -0,0 +1,777 @@ +/* + *

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551333#comment-15551333 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551249#comment-15551249 ] ASF GitHub Bot commented on FLINK-4348: --- Github user KurtYoung commented on a diff in the pull

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82134273 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -416,15 +333,14 @@ private boolean

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82132624 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -136,53 +127,34 @@ public

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551229#comment-15551229 ] ASF GitHub Bot commented on FLINK-4348: --- Github user KurtYoung commented on a diff in the pull

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551263#comment-15551263 ] ASF GitHub Bot commented on FLINK-4348: --- Github user KurtYoung commented on a diff in the pull

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82133014 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/stream/table/GroupWindowITCase.scala --- @@ -0,0 +1,777 @@ +/* + *

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551233#comment-15551233 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82134943 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/ResourceManager.java --- @@ -245,22 +255,26 @@ public TaskExecutorGateway

[jira] [Created] (FLINK-4752) Improve session window documentation

2016-10-06 Thread Till Rohrmann (JIRA)
Till Rohrmann created FLINK-4752: Summary: Improve session window documentation Key: FLINK-4752 URL: https://issues.apache.org/jira/browse/FLINK-4752 Project: Flink Issue Type: Improvement

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82140542 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/stream/table/GroupWindowITCase.scala --- @@ -0,0 +1,777 @@ +/* + *

[GitHub] flink issue #2397: [FLINK-4439] Validate 'bootstrap.servers' config in flink...

2016-10-06 Thread gheo21
Github user gheo21 commented on the issue: https://github.com/apache/flink/pull/2397 Hi guys @StephanEwen @rmetzger , I've rebased once again the branch and fixed the merge conflicts. The travis build has two failed builds but one is connection reset by maven and the other one is

[jira] [Commented] (FLINK-4439) Error message KafkaConsumer08 when all 'bootstrap.servers' are invalid

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551201#comment-15551201 ] ASF GitHub Bot commented on FLINK-4439: --- Github user gheo21 commented on the issue:

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551460#comment-15551460 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82149847 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/Resolvable.scala --- @@ -0,0 +1,26 @@ +/* + * Licensed

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82149863 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/Resolvable.scala --- @@ -0,0 +1,26 @@ +/* + * Licensed

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82147739 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82151066 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/runtime/aggregate/AggregateReduceCombineFunction.scala --- @@ -0,0 +1,132 @@

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread mxm
Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82155453 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -118,9 +109,9 @@ public void

[jira] [Closed] (FLINK-4699) Convert Kafka TableSource/TableSink tests to unit tests

2016-10-06 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4699?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Timo Walther closed FLINK-4699. --- Resolution: Duplicate > Convert Kafka TableSource/TableSink tests to unit tests >

[GitHub] flink pull request #2600: [FLINK-4751] [futures] Add thenCombineAsync functi...

2016-10-06 Thread uce
Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2600#discussion_r82142194 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/impl/FlinkFuture.java --- @@ -232,6 +242,40 @@ public R recover(Throwable failure)

[GitHub] flink pull request #2600: [FLINK-4751] [futures] Add thenCombineAsync functi...

2016-10-06 Thread uce
Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2600#discussion_r82141826 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/impl/FlinkFuture.java --- @@ -190,7 +191,16 @@ public R recover(Throwable failure)

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551361#comment-15551361 ] ASF GitHub Bot commented on FLINK-4751: --- Github user uce commented on a diff in the pull request:

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551360#comment-15551360 ] ASF GitHub Bot commented on FLINK-4751: --- Github user uce commented on a diff in the pull request:

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551359#comment-15551359 ] ASF GitHub Bot commented on FLINK-4751: --- Github user uce commented on a diff in the pull request:

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551406#comment-15551406 ] ASF GitHub Bot commented on FLINK-4751: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #2600: [FLINK-4751] [futures] Add thenCombineAsync function to F...

2016-10-06 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/2600 Thanks for the review @uce. I will address your comments and then merge the PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82146791 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/operators.scala --- @@ -497,3 +501,95 @@ case class

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551476#comment-15551476 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551449#comment-15551449 ] ASF GitHub Bot commented on FLINK-4751: --- Github user uce commented on a diff in the pull request:

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551459#comment-15551459 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82151050 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/runtime/aggregate/AggregateReduceCombineFunction.scala --- @@ -0,0 +1,132 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82152829 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82147669 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82147461 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82151146 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/runtime/aggregate/AggregateReduceCombineFunction.scala --- @@ -0,0 +1,132 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82147584 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82144274 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/datastream/DataStreamAggregate.scala --- @@ -0,0 +1,261 @@

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

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

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551557#comment-15551557 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on a diff in the pull request:

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread mxm
Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82157384 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -136,53 +127,34 @@ public

[GitHub] flink pull request #2600: [FLINK-4751] [futures] Add thenCombineAsync functi...

2016-10-06 Thread uce
Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2600#discussion_r82148748 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/impl/FlinkFuture.java --- @@ -232,6 +244,40 @@ public R recover(Throwable failure)

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82150960 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/groupWindows.scala --- @@ -0,0 +1,242 @@ +/* + *

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82150850 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/logical/groupWindows.scala --- @@ -0,0 +1,242 @@ +/* + *

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551475#comment-15551475 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Anton Mushin (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551505#comment-15551505 ] Anton Mushin commented on FLINK-4604: - I have some problem. I added rule in

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551536#comment-15551536 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on a diff in the pull request:

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551532#comment-15551532 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on a diff in the pull request:

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread mxm
Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82155691 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -205,46 +177,64 @@ public void

[jira] [Created] (FLINK-4753) Kafka 0.8 connector's Periodic Offset Committer should synchronize on checkpoint lock

2016-10-06 Thread Tzu-Li (Gordon) Tai (JIRA)
Tzu-Li (Gordon) Tai created FLINK-4753: -- Summary: Kafka 0.8 connector's Periodic Offset Committer should synchronize on checkpoint lock Key: FLINK-4753 URL: https://issues.apache.org/jira/browse/FLINK-4753

[GitHub] flink pull request #2600: [FLINK-4751] [futures] Add thenCombineAsync functi...

2016-10-06 Thread uce
Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2600#discussion_r82142626 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/impl/FlinkFuture.java --- @@ -232,6 +244,40 @@ public R recover(Throwable failure)

[jira] [Commented] (FLINK-3932) Implement State Backend Security

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551433#comment-15551433 ] ASF GitHub Bot commented on FLINK-3932: --- Github user mxm commented on the issue:

[GitHub] flink issue #2589: FLINK-3932 State Backend Security

2016-10-06 Thread mxm
Github user mxm commented on the issue: https://github.com/apache/flink/pull/2589 Thanks for the PR @vijikarthi. The changes look good to me. --- 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

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551547#comment-15551547 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on a diff in the pull request:

[GitHub] flink pull request #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread mxm
Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/2571#discussion_r82156547 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/slotmanager/SlotManager.java --- @@ -416,15 +333,14 @@ private boolean

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551417#comment-15551417 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink issue #2571: [FLINK-4348] Simplify logic of SlotManager

2016-10-06 Thread mxm
Github user mxm commented on the issue: https://github.com/apache/flink/pull/2571 Thanks for the comments @KurtYoung. I've updated the PR. No worries, four eyes always see more than two and it is natural that ideas progress over time :) --- If your project is set up for it, you can

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551563#comment-15551563 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on the issue:

[jira] [Commented] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552086#comment-15552086 ] Timo Walther commented on FLINK-4604: - Maybe the cost function of {{DataSetAggregate}} is faulty.

[jira] [Created] (FLINK-4756) NullPointerException on submiting a job with StreamExecutionEnvironment.createRemoteEnvironment to a flink cluster

2016-10-06 Thread Ruwen Moos (JIRA)
Ruwen Moos created FLINK-4756: - Summary: NullPointerException on submiting a job with StreamExecutionEnvironment.createRemoteEnvironment to a flink cluster Key: FLINK-4756 URL:

[GitHub] flink pull request #2602: [hotfix] Re-register restored processing time time...

2016-10-06 Thread kl0u
GitHub user kl0u opened a pull request: https://github.com/apache/flink/pull/2602 [hotfix] Re-register restored processing time timers. You can merge this pull request into a Git repository by running: $ git pull https://github.com/kl0u/flink hotfix_reregister Alternatively

[GitHub] flink pull request #2603: [FLINK-4745] [table] Convert KafkaTableSource test...

2016-10-06 Thread twalthr
GitHub user twalthr opened a pull request: https://github.com/apache/flink/pull/2603 [FLINK-4745] [table] Convert KafkaTableSource test to unit tests This PR replaces IT cases with unit tests. You can merge this pull request into a Git repository by running: $ git pull

[jira] [Commented] (FLINK-4745) Convert KafkaTableSource test to unit tests

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551679#comment-15551679 ] ASF GitHub Bot commented on FLINK-4745: --- GitHub user twalthr opened a pull request:

[jira] [Commented] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551706#comment-15551706 ] Timo Walther commented on FLINK-4604: - Your implementation looks good so far. The output of

[jira] [Closed] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-4751. Resolution: Fixed Added via 98710ead59305fc067cb8fdfab2c47d3bdc2e3fc > Extend Flink's futures to

[jira] [Comment Edited] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551943#comment-15551943 ] Timo Walther edited comment on FLINK-4604 at 10/6/16 1:36 PM: -- Ok, it seems

[jira] [Comment Edited] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Anton Mushin (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552063#comment-15552063 ] Anton Mushin edited comment on FLINK-4604 at 10/6/16 2:24 PM: -- bq. Can you

[jira] [Updated] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Anton Mushin (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anton Mushin updated FLINK-4604: Attachment: 1.jpg bq. Can you check if AggregateReduceFunctionsRule.matches is called? yes it is

[GitHub] flink pull request #2600: [FLINK-4751] [futures] Add thenCombineAsync functi...

2016-10-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2600 --- 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 is

[jira] [Commented] (FLINK-4751) Extend Flink's futures to support combining two futures

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4751?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551688#comment-15551688 ] ASF GitHub Bot commented on FLINK-4751: --- Github user asfgit closed the pull request at:

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551712#comment-15551712 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[jira] [Commented] (FLINK-4691) Add group-windows for streaming tables

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551734#comment-15551734 ] ASF GitHub Bot commented on FLINK-4691: --- Github user twalthr commented on a diff in the pull

[GitHub] flink pull request #2583: [FLINK-4730] Introduce checkpoint meta data

2016-10-06 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/2583#discussion_r82174054 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/CheckpointMetaData.java --- @@ -0,0 +1,113 @@ +/* + * Licensed to

[jira] [Comment Edited] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Anton Mushin (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551768#comment-15551768 ] Anton Mushin edited comment on FLINK-4604 at 10/6/16 12:22 PM: --- I used

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82177134 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/runtime/aggregate/AggregateReduceCombineFunction.scala --- @@ -0,0 +1,132 @@

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82177163 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/runtime/aggregate/AggregateReduceCombineFunction.scala --- @@ -0,0 +1,132 @@

[jira] [Commented] (FLINK-4348) Implement slot allocation protocol with TaskExecutor

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551911#comment-15551911 ] ASF GitHub Bot commented on FLINK-4348: --- Github user mxm commented on the issue:

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82186936 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/scala/table/expressionDsl.scala --- @@ -325,33 +337,42 @@ trait

[jira] [Commented] (FLINK-4604) Add support for standard deviation/variance

2016-10-06 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551943#comment-15551943 ] Timo Walther commented on FLINK-4604: - Ok, it seems that the {{AggregateReduceFunctionsRule}} does

[GitHub] flink pull request #2562: [FLINK-4691] [table] Add group-windows for streami...

2016-10-06 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r82170078 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/windows.scala --- @@ -0,0 +1,340 @@ +/* + * Licensed to the Apache

  1   2   >