[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268593#comment-16268593 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268586#comment-16268586 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268595#comment-16268595 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268587#comment-16268587 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268596#comment-16268596 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268592#comment-16268592 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268597#comment-16268597 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268588#comment-16268588 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268591#comment-16268591 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268589#comment-16268589 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268594#comment-16268594 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268590#comment-16268590 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153456278 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/RemoteInputChannel.java --- @@ -230,7 +230,7 @@ void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153457926 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -208,8 +244,79 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153455324 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -208,8 +244,79 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153455238 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -208,8 +244,79 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153460586 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyMessage.java --- @@ -584,4 +587,60 @@ static CloseRequest

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153454649 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -208,8 +244,79 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153448085 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/NettyMessageSerializationTest.java --- @@ -158,6 +158,15 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153456963 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -145,19 +171,29 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153447906 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyMessage.java --- @@ -584,4 +587,60 @@ static CloseRequest

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153456919 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -122,21 +136,33 @@ public void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153446896 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/CreditBasedClientHandler.java --- @@ -102,13 +102,21 @@ void

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153457551 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestClientHandlerTest.java --- @@ -208,8 +244,79 @@ public void

[jira] [Assigned] (FLINK-8150) WebUI in FLIP-6 mode exposes TaskManagerConnection IDs as TaskManager IDs

2017-11-28 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann reassigned FLINK-8150: Assignee: Till Rohrmann > WebUI in FLIP-6 mode exposes TaskManagerConnection IDs as

[jira] [Commented] (FLINK-8104) Fix Row value constructor

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8104?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268553#comment-16268553 ] ASF GitHub Bot commented on FLINK-8104: --- Github user twalthr commented on the issue:

[GitHub] flink issue #5040: [FLINK-8104][Table API] fixing ROW type value constructor...

2017-11-28 Thread twalthr
Github user twalthr commented on the issue: https://github.com/apache/flink/pull/5040 Thanks. I will have a look at it soon. I think it ok that the Table API is a super set of features in this case. ---

[jira] [Commented] (FLINK-4812) Report Watermark metrics in all operators

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268547#comment-16268547 ] ASF GitHub Bot commented on FLINK-4812: --- GitHub user zentol opened a pull request:

[GitHub] flink pull request #5092: [FLINK-4812][metrics] Expose currentLowWatermark f...

2017-11-28 Thread zentol
GitHub user zentol opened a pull request: https://github.com/apache/flink/pull/5092 [FLINK-4812][metrics] Expose currentLowWatermark for all operators ## What is the purpose of the change This PR makes all operators expose the current low watermark through the metric

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

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

[GitHub] flink issue #5063: [FLINK-8144] [table] Optimize the timer logic in RowTimeU...

2017-11-28 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5063 Great, thank you! ---

[jira] [Commented] (FLINK-8143) Port SubtaskMetricsHandler to new REST endpoint

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

[GitHub] flink pull request #5082: [FLINK-8143][flip6] Migrate SubtaskMetricsHandler ...

2017-11-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/5082#discussion_r153450679 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/SubtaskIndexPathParameter.java --- @@ -0,0 +1,47 @@ +/* + *

[GitHub] flink pull request #5070: [FLINK-8151][table]Remove Map type equality compar...

2017-11-28 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5070#discussion_r153449102 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/expressions/MapTypeTest.scala --- @@ -139,13 +139,6 @@ class MapTypeTest

[jira] [Commented] (FLINK-8151) [Table] removing map value equality check

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

[GitHub] flink pull request #5070: [FLINK-8151][table]Remove Map type equality compar...

2017-11-28 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5070#discussion_r153448795 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/calls/ScalarOperators.scala --- @@ -188,13 +188,6 @@ object

[jira] [Commented] (FLINK-8151) [Table] removing map value equality check

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

[jira] [Commented] (FLINK-7717) Port TaskManagerMetricsHandler to new REST endpoint

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

[GitHub] flink pull request #5081: [FLINK-7717][flip6] Migrate TaskManagerMetricsHand...

2017-11-28 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/5081#discussion_r153448441 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/metrics/TaskManagerMetricsHandler.java --- @@ -0,0 +1,64 @@ +/*

[jira] [Commented] (FLINK-8161) Flakey YARNSessionCapacitySchedulerITCase on Travis

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268500#comment-16268500 ] ASF GitHub Bot commented on FLINK-8161: --- Github user tillrohrmann closed the pull request at:

[jira] [Closed] (FLINK-8161) Flakey YARNSessionCapacitySchedulerITCase on Travis

2017-11-28 Thread Till Rohrmann (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8161?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Till Rohrmann closed FLINK-8161. Resolution: Fixed Fixed via 7f0c8344c3bd066085b9aa1564849d7716b1ec7e > Flakey

[GitHub] flink pull request #5085: [FLINK-8161] [tests] Harden YARNSessionCapacitySch...

2017-11-28 Thread tillrohrmann
Github user tillrohrmann closed the pull request at: https://github.com/apache/flink/pull/5085 ---

[GitHub] flink pull request #4533: [FLINK-7416][network] Implement Netty receiver out...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4533#discussion_r153445970 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/RemoteInputChannel.java --- @@ -283,10 +283,13 @@ public String

[jira] [Commented] (FLINK-8161) Flakey YARNSessionCapacitySchedulerITCase on Travis

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268496#comment-16268496 ] ASF GitHub Bot commented on FLINK-8161: --- Github user tillrohrmann commented on the issue:

[jira] [Commented] (FLINK-7416) Implement Netty receiver outgoing pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268497#comment-16268497 ] ASF GitHub Bot commented on FLINK-7416: --- Github user NicoK commented on a diff in the pull request:

[GitHub] flink issue #5085: [FLINK-8161] [tests] Harden YARNSessionCapacitySchedulerI...

2017-11-28 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/5085 Merging. ---

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268495#comment-16268495 ] ASF GitHub Bot commented on FLINK-8144: --- Github user dianfu commented on the issue:

[GitHub] flink issue #5063: [FLINK-8144] [table] Optimize the timer logic in RowTimeU...

2017-11-28 Thread dianfu
Github user dianfu commented on the issue: https://github.com/apache/flink/pull/5063 The current known impact is that it will need to checkpoint more timer. For the impact to performance, I will do some benchmark later. :) I think we only need to consider this optimization if

[jira] [Commented] (FLINK-8158) Rowtime window inner join emits late data

2017-11-28 Thread Hequn Cheng (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268494#comment-16268494 ] Hequn Cheng commented on FLINK-8158: OK, I will make a pr with test cases to reproduce the problem. >

[jira] [Commented] (FLINK-7903) Add Flip6 build profile

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7903?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268478#comment-16268478 ] ASF GitHub Bot commented on FLINK-7903: --- Github user tillrohrmann commented on the issue:

[GitHub] flink issue #4889: [FLINK-7903] [tests] Add flip6 build profile

2017-11-28 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/4889 Thanks for the review @zentol. Rebasing and once Travis gives green light, I'll merge it. ---

[jira] [Commented] (FLINK-7406) Implement Netty receiver incoming pipeline for credit-based

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268475#comment-16268475 ] ASF GitHub Bot commented on FLINK-7406: --- Github user NicoK commented on a diff in the pull request:

[GitHub] flink pull request #4509: [FLINK-7406][network] Implement Netty receiver inc...

2017-11-28 Thread NicoK
Github user NicoK commented on a diff in the pull request: https://github.com/apache/flink/pull/4509#discussion_r153441043 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/RemoteInputChannel.java --- @@ -306,12 +306,10 @@ public void

[jira] [Assigned] (FLINK-8163) NonHAQueryableStateFsBackendITCase test getting stuck on Travis

2017-11-28 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kostas Kloudas reassigned FLINK-8163: - Assignee: Kostas Kloudas > NonHAQueryableStateFsBackendITCase test getting stuck on

[GitHub] flink issue #4581: [FLINK-7499][io] fix double buffer release in SpillableSu...

2017-11-28 Thread NicoK
Github user NicoK commented on the issue: https://github.com/apache/flink/pull/4581 test failures are unrelated this time (failing kafka download for the end-to-end tests) ---

[jira] [Created] (FLINK-8163) NonHAQueryableStateFsBackendITCase test getting stuck on Travis

2017-11-28 Thread Till Rohrmann (JIRA)
Till Rohrmann created FLINK-8163: Summary: NonHAQueryableStateFsBackendITCase test getting stuck on Travis Key: FLINK-8163 URL: https://issues.apache.org/jira/browse/FLINK-8163 Project: Flink

[jira] [Commented] (FLINK-7499) double buffer release in SpillableSubpartitionView

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268467#comment-16268467 ] ASF GitHub Bot commented on FLINK-7499: --- Github user NicoK commented on the issue:

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

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

[GitHub] flink issue #5063: [FLINK-8144] [table] Optimize the timer logic in RowTimeU...

2017-11-28 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5063 Thank you @dianfu for the discussion. I raised concerns about this PR because it seem to me to optimize for a special use case and possibly cause regression for more common workloads. However,

[jira] [Commented] (FLINK-7873) Introduce CheckpointCacheManager for reading checkpoint data locally when performing failover

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

[GitHub] flink pull request #5074: [FLINK-7873] [runtime] Introduce local recovery

2017-11-28 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/5074#discussion_r153432998 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/CachedCheckpointStreamFactory.java --- @@ -0,0 +1,195 @@ +/* + *

[jira] [Commented] (FLINK-7873) Introduce CheckpointCacheManager for reading checkpoint data locally when performing failover

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

[GitHub] flink pull request #5068: [FLINK-8122] [table] Name all built-in table sinks...

2017-11-28 Thread xccui
Github user xccui commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r153432702 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/sources/TableSource.scala --- @@ -60,4 +60,14 @@ trait TableSource[T] {

[jira] [Commented] (FLINK-8122) Name all table sinks and sources

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268430#comment-16268430 ] ASF GitHub Bot commented on FLINK-8122: --- Github user xccui commented on a diff in the pull request:

[jira] [Commented] (FLINK-8106) Optimize the timer logic in AbstractKeyedCEPPatternOperator

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268427#comment-16268427 ] ASF GitHub Bot commented on FLINK-8106: --- Github user dianfu commented on the issue:

[GitHub] flink pull request #5074: [FLINK-7873] [runtime] Introduce local recovery

2017-11-28 Thread StefanRRichter
Github user StefanRRichter commented on a diff in the pull request: https://github.com/apache/flink/pull/5074#discussion_r153432415 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/state/CachedCheckpointStreamFactory.java --- @@ -0,0 +1,195 @@ +/* + *

[jira] [Commented] (FLINK-8106) Optimize the timer logic in AbstractKeyedCEPPatternOperator

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268428#comment-16268428 ] ASF GitHub Bot commented on FLINK-8106: --- Github user dianfu closed the pull request at:

[jira] [Closed] (FLINK-8106) Optimize the timer logic in AbstractKeyedCEPPatternOperator

2017-11-28 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8106?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu closed FLINK-8106. -- Resolution: Not A Problem > Optimize the timer logic in AbstractKeyedCEPPatternOperator >

[GitHub] flink pull request #5036: [FLINK-8106] [cep] Optimize the timer logic in Abs...

2017-11-28 Thread dianfu
Github user dianfu closed the pull request at: https://github.com/apache/flink/pull/5036 ---

[GitHub] flink issue #5036: [FLINK-8106] [cep] Optimize the timer logic in AbstractKe...

2017-11-28 Thread dianfu
Github user dianfu commented on the issue: https://github.com/apache/flink/pull/5036 Close this PR for the same reason as #5063 ---

[jira] [Closed] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

2017-11-28 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu closed FLINK-8144. -- Resolution: Not A Problem > Optimize the timer logic in RowTimeUnboundedOver >

[GitHub] flink issue #5063: [FLINK-8144] [table] Optimize the timer logic in RowTimeU...

2017-11-28 Thread dianfu
Github user dianfu commented on the issue: https://github.com/apache/flink/pull/5063 For historical reasons, we only use punctuated watermark currently. But I think you are right, we should consider using periodic watermark instead. Thanks a lot for your explanation. I will close

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268419#comment-16268419 ] ASF GitHub Bot commented on FLINK-8144: --- Github user dianfu commented on the issue:

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268420#comment-16268420 ] ASF GitHub Bot commented on FLINK-8144: --- Github user dianfu closed the pull request at:

[GitHub] flink pull request #5063: [FLINK-8144] [table] Optimize the timer logic in R...

2017-11-28 Thread dianfu
Github user dianfu closed the pull request at: https://github.com/apache/flink/pull/5063 ---

[jira] [Commented] (FLINK-8122) Name all table sinks and sources

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

[GitHub] flink pull request #5068: [FLINK-8122] [table] Name all built-in table sinks...

2017-11-28 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r153430172 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/sources/TableSource.scala --- @@ -60,4 +60,14 @@ trait TableSource[T] {

[jira] [Commented] (FLINK-8158) Rowtime window inner join emits late data

2017-11-28 Thread Fabian Hueske (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268409#comment-16268409 ] Fabian Hueske commented on FLINK-8158: -- Hi [~hequn8128], thanks for looking into this! Can you

[jira] [Commented] (FLINK-4822) Ensure that the Kafka 0.8 connector is compatible with kafka-consumer-groups.sh

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

[GitHub] flink pull request #5050: [FLINK-4822] Ensure that the Kafka 0.8 connector i...

2017-11-28 Thread taizilongxu
Github user taizilongxu commented on a diff in the pull request: https://github.com/apache/flink/pull/5050#discussion_r153426741 --- Diff: flink-connectors/flink-connector-kafka-0.8/src/main/java/org/apache/flink/streaming/connectors/kafka/internals/ZookeeperOffsetHandler.java ---

[GitHub] flink issue #5063: [FLINK-8144] [table] Optimize the timer logic in RowTimeU...

2017-11-28 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5063 I assume you had also tried periodic watermarks in your setting. I'm curious why they didn't work for you. A periodic watermark assigner extracts the timestamp from each record and just

[jira] [Commented] (FLINK-8144) Optimize the timer logic in RowTimeUnboundedOver

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

[GitHub] flink issue #5047: Code refine of WordWithCount

2017-11-28 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5047 IMO, the added comment might rather confuse readers than help them. ---

[jira] [Commented] (FLINK-7499) double buffer release in SpillableSubpartitionView

2017-11-28 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16268374#comment-16268374 ] ASF GitHub Bot commented on FLINK-7499: --- Github user NicoK commented on the issue:

[GitHub] flink issue #4581: [FLINK-7499][io] fix double buffer release in SpillableSu...

2017-11-28 Thread NicoK
Github user NicoK commented on the issue: https://github.com/apache/flink/pull/4581 sorry about that - tested locally now and it should go through...we'll see ---

<    1   2