[jira] [Commented] (FLINK-6311) NPE in FlinkKinesisConsumer if source was closed before run

2017-04-18 Thread mingleizhang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15974124#comment-15974124 ] mingleizhang commented on FLINK-6311: - [~tzulitai] Thanks for telling me so useful information. I am

[jira] [Commented] (FLINK-5095) Add explicit notifyOfAddedX methods to MetricReporter interface

2017-04-18 Thread Bowen Li (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5095?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15974103#comment-15974103 ] Bowen Li commented on FLINK-5095: - I'll spend time in the following few days to see how other projects

[GitHub] flink issue #3736: [Flink-6013][metrics] Add Datadog HTTP metrics reporter

2017-04-18 Thread bowenli86
Github user bowenli86 commented on the issue: https://github.com/apache/flink/pull/3736 Addressed @zentol 's comments. Ready for another round. Thanks! --- 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

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread bowenli86
Github user bowenli86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112115930 --- Diff: flink-metrics/flink-metrics-datadog/pom.xml --- @@ -0,0 +1,85 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread bowenli86
Github user bowenli86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112115925 --- Diff: docs/monitoring/metrics.md --- @@ -436,6 +436,38 @@ metrics.reporter.stsd.port: 8125 {% endhighlight %} +### Datadog

[jira] [Commented] (FLINK-6311) NPE in FlinkKinesisConsumer if source was closed before run

2017-04-18 Thread Tzu-Li (Gordon) Tai (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15974069#comment-15974069 ] Tzu-Li (Gordon) Tai commented on FLINK-6311: the {{flink-connector-kinesis}} module is only

[jira] [Assigned] (FLINK-6311) NPE in FlinkKinesisConsumer if source was closed before run

2017-04-18 Thread mingleizhang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] mingleizhang reassigned FLINK-6311: --- Assignee: mingleizhang > NPE in FlinkKinesisConsumer if source was closed before run >

[jira] [Commented] (FLINK-6311) NPE in FlinkKinesisConsumer if source was closed before run

2017-04-18 Thread mingleizhang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15974012#comment-15974012 ] mingleizhang commented on FLINK-6311: - [~tzulitai] I just watch FlinkKinesisConsumer which under the

[jira] [Closed] (FLINK-6149) add additional flink logical relation nodes

2017-04-18 Thread Kurt Young (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6149?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kurt Young closed FLINK-6149. - Resolution: Implemented Fix Version/s: 1.3.0 > add additional flink logical relation nodes >

[GitHub] flink pull request #3594: [FLINK-6149] [table] Add additional flink logical ...

2017-04-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3594 --- 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-6149) add additional flink logical relation nodes

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

[jira] [Commented] (FLINK-4821) Implement rescalable non-partitioned state for Kinesis Connector

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

[GitHub] flink issue #3001: [FLINK-4821] [kinesis] Implement rescalable non-partition...

2017-04-18 Thread tony810430
Github user tony810430 commented on the issue: https://github.com/apache/flink/pull/3001 Hi @tzulitai , I have rebased it and updated to the public API. --- 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

[jira] [Commented] (FLINK-6232) Support proctime inner equi-join between two streams in the SQL API

2017-04-18 Thread hongyuhong (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15973982#comment-15973982 ] hongyuhong commented on FLINK-6232: --- Hi [~fhueske] i have implement it use normal timestamp attribute

[GitHub] flink issue #3001: [FLINK-4821] [kinesis] Implement rescalable non-partition...

2017-04-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3001 Hi @tony810430, The exposure for union list state was just merged to master. Could you rebase this? Once rebased I'll do a full review so that we can finally work towards merging this :)

[jira] [Commented] (FLINK-6250) Distinct procTime with Rows boundaries

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

[jira] [Commented] (FLINK-6250) Distinct procTime with Rows boundaries

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

[GitHub] flink pull request #3732: [FLINK-6250] Distinct procTime with Rows boundarie...

2017-04-18 Thread hongyuhong
Github user hongyuhong commented on a diff in the pull request: https://github.com/apache/flink/pull/3732#discussion_r112103318 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/ProcTimeBoundedDistinctRowsOver.scala --- @@ -0,0 +1,238

[GitHub] flink pull request #3732: [FLINK-6250] Distinct procTime with Rows boundarie...

2017-04-18 Thread hongyuhong
Github user hongyuhong commented on a diff in the pull request: https://github.com/apache/flink/pull/3732#discussion_r112105763 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/sql/SqlITCase.scala --- @@ -1140,6 +1140,202 @@ class

[jira] [Commented] (FLINK-6250) Distinct procTime with Rows boundaries

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

[GitHub] flink pull request #3732: [FLINK-6250] Distinct procTime with Rows boundarie...

2017-04-18 Thread hongyuhong
Github user hongyuhong commented on a diff in the pull request: https://github.com/apache/flink/pull/3732#discussion_r112105029 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/ProcTimeBoundedDistinctRowsOver.scala --- @@ -0,0 +1,238

[jira] [Commented] (FLINK-6250) Distinct procTime with Rows boundaries

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

[GitHub] flink pull request #3732: [FLINK-6250] Distinct procTime with Rows boundarie...

2017-04-18 Thread hongyuhong
Github user hongyuhong commented on a diff in the pull request: https://github.com/apache/flink/pull/3732#discussion_r112105507 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/ProcTimeBoundedDistinctRowsOver.scala --- @@ -0,0 +1,238

[jira] [Resolved] (FLINK-6324) Refine state access methods in OperatorStateStore

2017-04-18 Thread Tzu-Li (Gordon) Tai (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6324?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tzu-Li (Gordon) Tai resolved FLINK-6324. Resolution: Fixed Fix Version/s: 1.3.0 Resolved for 1.3.0 with

[jira] [Resolved] (FLINK-5991) Expose Broadcast Operator State through public APIs

2017-04-18 Thread Tzu-Li (Gordon) Tai (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-5991?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tzu-Li (Gordon) Tai resolved FLINK-5991. Resolution: Fixed Resolved for 1.3.0 with

[jira] [Commented] (FLINK-5991) Expose Broadcast Operator State through public APIs

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

[GitHub] flink pull request #3508: [FLINK-5991] [state-backend, streaming] Expose Bro...

2017-04-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3508 --- 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] [Comment Edited] (FLINK-6130) Consider calling resourceManager#getTerminationFuture() with lock held

2017-04-18 Thread mingleizhang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15973937#comment-15973937 ] mingleizhang edited comment on FLINK-6130 at 4/19/17 2:36 AM: -- [~Zentol]

[jira] [Commented] (FLINK-6130) Consider calling resourceManager#getTerminationFuture() with lock held

2017-04-18 Thread mingleizhang (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-6130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15973937#comment-15973937 ] mingleizhang commented on FLINK-6130: - [~Zentol] [~till.rohrmann] That makes sense to me now. So, I

[jira] [Created] (FLINK-6324) Refine state access methods in OperatorStateStore

2017-04-18 Thread Tzu-Li (Gordon) Tai (JIRA)
Tzu-Li (Gordon) Tai created FLINK-6324: -- Summary: Refine state access methods in OperatorStateStore Key: FLINK-6324 URL: https://issues.apache.org/jira/browse/FLINK-6324 Project: Flink

[jira] [Commented] (FLINK-6149) add additional flink logical relation nodes

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

[GitHub] flink issue #3594: [FLINK-6149] [table] Add additional flink logical relatio...

2017-04-18 Thread KurtYoung
Github user KurtYoung commented on the issue: https://github.com/apache/flink/pull/3594 Rebased to master and will merge after build check --- 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-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink issue #3735: [FLINK-6242] [table] Add code generation for DataSet Aggr...

2017-04-18 Thread shaoxuan-wang
Github user shaoxuan-wang commented on the issue: https://github.com/apache/flink/pull/3735 @fhueske thanks for your feedback. Yes, we could keep GeneratedAggregations interface very clean as ``` abstract class GeneratedAggregations extends Function { def

[jira] [Commented] (FLINK-6295) use LoadingCache instead of WeakHashMap to lower latency

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

[GitHub] flink issue #3709: [FLINK-6295]use LoadingCache instead of WeakHashMap to lo...

2017-04-18 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the issue: https://github.com/apache/flink/pull/3709 Ok i think i've got your point. Now using WeakHashMap, we add entries when the map doesn't contain the requested EG id, remove invalid entries when GC happens. By adding

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread bowenli86
Github user bowenli86 commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112083224 --- Diff: flink-metrics/flink-metrics-datadog/src/main/java/org/apache/flink/metrics/datadog/metric/DGauge.java --- @@ -0,0 +1,30 @@ +/* + *

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112080959 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/RetractionITCase.scala --- @@ -0,0 +1,310 @@ +/* + *

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112081457 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/RetractionITCase.scala --- @@ -0,0 +1,310 @@ +/* + *

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112078731 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -41,14 +41,16 @@ class

[jira] [Commented] (FLINK-6091) Implement and turn on the retraction for aggregates

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

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112078707 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -41,14 +41,16 @@ class

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112080186 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -84,16 +109,38 @@ class

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112065725 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/datastream/DataStreamOverAggregate.scala --- @@ -129,11 +131,17 @@

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112079294 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -84,16 +109,38 @@ class

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112078358 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GroupAggProcessFunction.scala --- @@ -68,12 +70,35 @@ class

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112065233 --- Diff: flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/RowSerializer.java --- @@ -152,6 +161,7 @@ public void serialize(Row record,

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112081928 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/stream/RetractionITCase.scala --- @@ -0,0 +1,310 @@ +/* + *

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112064885 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/nodes/datastream/DataStreamGroupWindowAggregate.scala --- @@ -107,6

[GitHub] flink pull request #3733: [FLINK-6091] [table] Implement and turn on retract...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3733#discussion_r112060500 --- Diff: flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/RowSerializer.java --- @@ -152,6 +161,7 @@ public void serialize(Row record,

[jira] [Commented] (FLINK-6032) CEP-Clean up the operator state when not needed.

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

[GitHub] flink pull request #3541: [FLINK-6032] [cep] Clean-up operator state when no...

2017-04-18 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/flink/pull/3541#discussion_r112071560 --- Diff: flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/SubEvent.java --- @@ -31,6 +31,18 @@ public double getVolume() { }

[jira] [Commented] (FLINK-6032) CEP-Clean up the operator state when not needed.

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

[GitHub] flink pull request #3541: [FLINK-6032] [cep] Clean-up operator state when no...

2017-04-18 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/flink/pull/3541#discussion_r112071274 --- Diff: flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/operator/AbstractKeyedCEPPatternOperator.java --- @@ -385,4 +393,25 @@ public int

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112065478 --- Diff: flink-metrics/flink-metrics-datadog/pom.xml --- @@ -0,0 +1,85 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112065330 --- Diff: docs/monitoring/metrics.md --- @@ -436,6 +436,38 @@ metrics.reporter.stsd.port: 8125 {% endhighlight %} +### Datadog

[GitHub] flink pull request #3736: [Flink-6013][metrics] Add Datadog HTTP metrics rep...

2017-04-18 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/3736#discussion_r112065890 --- Diff: flink-metrics/flink-metrics-datadog/src/main/java/org/apache/flink/metrics/datadog/metric/DGauge.java --- @@ -0,0 +1,30 @@ +/* + *

[GitHub] flink issue #3736: [Flink-6013][metrics] Add Datadog HTTP metrics reporter

2017-04-18 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/3736 If you want to enforce a specific scope config then i would suggest to instead ignore the configured one (which means not using `MetricGroup#getMetricIdentifier()`) and instead extracting the

[GitHub] flink issue #3736: [Flink-6013][metrics] Add Datadog HTTP metrics reporter

2017-04-18 Thread bowenli86
Github user bowenli86 commented on the issue: https://github.com/apache/flink/pull/3736 Great advice. Thanks, @zentol ! My questions to your suggestions: 1. If I make DatadogHttpReport implement MetricReporter interface, shall I just ignore Histogram and Meter which are

[GitHub] flink issue #3737: Release 1.2

2017-04-18 Thread kimlng
Github user kimlng commented on the issue: https://github.com/apache/flink/pull/3737 Pulled request accidentally. Need to revert this. --- 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

[GitHub] flink pull request #3737: Release 1.2

2017-04-18 Thread kimlng
Github user kimlng closed the pull request at: https://github.com/apache/flink/pull/3737 --- 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

[GitHub] flink pull request #3737: Release 1.2

2017-04-18 Thread kimlng
GitHub user kimlng opened a pull request: https://github.com/apache/flink/pull/3737 Release 1.2 Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration. If your changes take all of the items into

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112002278 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetSlideTimeWindowAggReduceGroupFunction.scala --- @@

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r111986866 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/CodeGenerator.scala --- @@ -263,33 +263,56 @@ class CodeGenerator(

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112002408 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetSlideTimeWindowAggReduceGroupFunction.scala --- @@

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r111990633 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/CodeGenerator.scala --- @@ -263,33 +263,56 @@ class CodeGenerator(

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112001966 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetSlideWindowAggReduceCombineFunction.scala --- @@

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112003763 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetWindowAggMapFunction.scala --- @@ -25,58 +25,56 @@

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112007816 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/GeneratedAggregations.scala --- @@ -36,6 +36,15 @@ abstract

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r112002140 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetSlideWindowAggReduceCombineFunction.scala --- @@

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r111995370 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetAggFunction.scala --- @@ -68,23 +72,16 @@ class

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r111995303 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/DataSetFinalAggFunction.scala --- @@ -19,88 +19,71 @@

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[GitHub] flink pull request #3735: [FLINK-6242] [table] Add code generation for DataS...

2017-04-18 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/3735#discussion_r111991648 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/CodeGenerator.scala --- @@ -451,13 +581,34 @@ class CodeGenerator(

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

[jira] [Commented] (FLINK-6242) codeGen DataSet Goupingwindow Aggregates

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

  1   2   3   4   5   >