[GitHub] flink pull request #5132: [FLINK-8203] [FLINK-7681] [table] Make schema defi...

2017-12-07 Thread twalthr
GitHub user twalthr opened a pull request: https://github.com/apache/flink/pull/5132 [FLINK-8203] [FLINK-7681] [table] Make schema definition of DataStream/DataSet to Table conversion more flexible ## What is the purpose of the change This PR makes the schema definition

[jira] [Commented] (FLINK-8089) Fulfil slot requests with unused offered slots

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

[GitHub] flink pull request #5090: [FLINK-8089] Also check for other pending slot req...

2017-12-07 Thread GJL
Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/5090#discussion_r155495322 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/instance/SlotPoolTest.java --- @@ -383,6 +386,76 @@ public void

[GitHub] flink pull request #5047: Code refine of WordWithCount

2017-12-07 Thread harborl
Github user harborl closed the pull request at: https://github.com/apache/flink/pull/5047 ---

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155490712 --- Diff: flink-connectors/flink-orc/src/test/java/org/apache/flink/orc/OrcTableSourceTest.java --- @@ -155,11 +160,14 @@ public void testProjectFields()

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

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

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

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

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

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

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

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

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

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

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

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

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

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281718#comment-16281718 ] 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-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155491207 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/sinks/TableSink.scala --- @@ -45,6 +45,10 @@ trait TableSink[T] { /**

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155493818 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/stream/table/TableSinkITCase.scala --- @@ -583,7 +583,7 @@

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155489738 --- Diff: flink-connectors/flink-connector-kafka-base/src/test/java/org/apache/flink/streaming/connectors/kafka/KafkaTableSinkTestBase.java --- @@ -54,6

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155492981 --- Diff: flink-connectors/flink-connector-cassandra/src/main/java/org/apache/flink/streaming/connectors/cassandra/CassandraAppendTableSink.java --- @@

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155493746 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/batch/table/TableSourceITCase.scala --- @@ -58,7 +58,7 @@ class

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

2017-12-07 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5068#discussion_r155489914 --- Diff: flink-connectors/flink-connector-kafka-base/src/test/java/org/apache/flink/streaming/connectors/kafka/KafkaTableSourceTestBase.java --- @@ -62,13

[jira] [Assigned] (FLINK-7681) Support arbitrary TypeInformation in Table API

2017-12-07 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Timo Walther reassigned FLINK-7681: --- Assignee: Timo Walther > Support arbitrary TypeInformation in Table API >

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

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

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281648#comment-16281648 ] ASF GitHub Bot commented on FLINK-8133: --- Github user zentol commented on the issue:

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482395 --- Diff: flink-docs/README.md --- @@ -0,0 +1,36 @@ + + +# Documentation generators + +This module contains generators that create HTML

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482662 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java --- @@ -317,7 +317,21 @@ public int compare(

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482491 --- Diff: flink-docs/README.md --- @@ -0,0 +1,36 @@ + + +# Documentation generators + +This module contains generators that create HTML

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155483634 --- Diff: flink-docs/src/main/java/org/apache/flink/docs/rest/RestAPIDocGenerator.java --- @@ -0,0 +1,315 @@ +/* + * Licensed to the Apache Software

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155483740 --- Diff: flink-docs/src/main/java/org/apache/flink/docs/rest/RestAPIDocGenerator.java --- @@ -0,0 +1,315 @@ +/* + * Licensed to the Apache Software

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482310 --- Diff: flink-docs/README.md --- @@ -0,0 +1,36 @@ + + +# Documentation generators + +This module contains generators that create HTML

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482231 --- Diff: flink-docs/README.md --- @@ -0,0 +1,36 @@ + + +# Documentation generators + +This module contains generators that create HTML

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155482058 --- Diff: flink-docs/README.md --- @@ -0,0 +1,36 @@ + + +# Documentation generators + +This module contains generators that create HTML

[GitHub] flink pull request #5052: [FLINK-8133][REST][docs] Generate REST API documen...

2017-12-07 Thread kl0u
Github user kl0u commented on a diff in the pull request: https://github.com/apache/flink/pull/5052#discussion_r155483284 --- Diff: flink-docs/src/main/java/org/apache/flink/docs/rest/RestAPIDocGenerator.java --- @@ -0,0 +1,315 @@ +/* + * Licensed to the Apache Software

[GitHub] flink issue #5052: [FLINK-8133][REST][docs] Generate REST API documentation

2017-12-07 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/5052 Since you found no errors I would like to merge it as is (including the changes to `rest_api.md`) and iterate upon it in follow-ups. My goal was not to provide a picture-perfect documentation, but to

[jira] [Commented] (FLINK-8139) Check for proper equals() and hashCode() when registering a table

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

[GitHub] flink issue #5065: [FLINK-8139][table] Check for proper equals() and hashCod...

2017-12-07 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5065 Hmm, I agree with @twalthr. Adding these checks when a DataSet/DataStream is imported would prevent valid queries (e.g., those that to simple transformations). I would not add a config switch to

[jira] [Comment Edited] (FLINK-8205) Multi key get

2017-12-07 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281624#comment-16281624 ] Kostas Kloudas edited comment on FLINK-8205 at 12/7/17 10:13 AM: - Actually

[jira] [Commented] (FLINK-8205) Multi key get

2017-12-07 Thread Kostas Kloudas (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281624#comment-16281624 ] Kostas Kloudas commented on FLINK-8205: --- Actually if you want to work on it, I would be more than

[jira] [Commented] (FLINK-8133) Generate documentation for new REST API

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281579#comment-16281579 ] ASF GitHub Bot commented on FLINK-8133: --- Github user kl0u commented on the issue:

[GitHub] flink issue #5052: [FLINK-8133][REST][docs] Generate REST API documentation

2017-12-07 Thread kl0u
Github user kl0u commented on the issue: https://github.com/apache/flink/pull/5052 From what I understand, this PR is just for the tooling, and not for the actual content. So the changes in the `test_api.md` are just a mock to see how it would look. If this is correct,

[jira] [Commented] (FLINK-7468) Implement sender backlog logic for credit-based

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281560#comment-16281560 ] ASF GitHub Bot commented on FLINK-7468: --- Github user zhijiangW commented on the issue:

[GitHub] flink issue #4559: [FLINK-7468][network] Implement sender backlog logic for ...

2017-12-07 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/4559 Or we make the backlog as `AtomicInteger` to keep the current process, otherwise we may need to call `decreaseStatistics` in different parts in the region of `synchronized(buffers)` . What do you

[jira] [Commented] (FLINK-8215) Collections codegen exception when constructing Array or Map via SQL API

2017-12-07 Thread Timo Walther (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281537#comment-16281537 ] Timo Walther commented on FLINK-8215: - I think the Table API should not support automatic type

[GitHub] flink issue #4559: [FLINK-7468][network] Implement sender backlog logic for ...

2017-12-07 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/4559 @NicoK @pnowojski , for backlog thread-safe issue, the current implementation is not thread-safe and we should restore my previous implementation that calls `decreaseStatistics` inside the method

[jira] [Commented] (FLINK-7468) Implement sender backlog logic for credit-based

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-7468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281505#comment-16281505 ] ASF GitHub Bot commented on FLINK-7468: --- Github user zhijiangW commented on the issue:

[jira] [Commented] (FLINK-7468) Implement sender backlog logic for credit-based

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

[GitHub] flink pull request #4559: [FLINK-7468][network] Implement sender backlog log...

2017-12-07 Thread zhijiangW
Github user zhijiangW commented on a diff in the pull request: https://github.com/apache/flink/pull/4559#discussion_r155458048 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/SpillableSubpartitionView.java --- @@ -145,6 +145,10 @@ public Buffer

[jira] [Commented] (FLINK-7468) Implement sender backlog logic for credit-based

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

[jira] [Commented] (FLINK-7468) Implement sender backlog logic for credit-based

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

[GitHub] flink pull request #4559: [FLINK-7468][network] Implement sender backlog log...

2017-12-07 Thread zhijiangW
Github user zhijiangW commented on a diff in the pull request: https://github.com/apache/flink/pull/4559#discussion_r155454935 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/ResultSubpartitionView.java --- @@ -22,32 +22,57 @@ import

[GitHub] flink pull request #4559: [FLINK-7468][network] Implement sender backlog log...

2017-12-07 Thread zhijiangW
Github user zhijiangW commented on a diff in the pull request: https://github.com/apache/flink/pull/4559#discussion_r155454886 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/PipelinedSubpartitionView.java --- @@ -39,13 +39,15 @@

[jira] [Commented] (FLINK-8218) move flink-connector-kinesis examples from /src to /test

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281480#comment-16281480 ] ASF GitHub Bot commented on FLINK-8218: --- GitHub user bowenli86 opened a pull request:

[GitHub] flink pull request #5131: [FLINK-8218] move flink-connector-kinesis examples...

2017-12-07 Thread bowenli86
GitHub user bowenli86 opened a pull request: https://github.com/apache/flink/pull/5131 [FLINK-8218] move flink-connector-kinesis examples from /src to /test ## What is the purpose of the change just saw there are runnable examples in src tree and traced back to this PR. I

[jira] [Commented] (FLINK-7749) Refactor the ResultPartitionWriter into an interface

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

[GitHub] flink issue #5069: [FLINK-8149][kinesis] Replace usages of deprecated Serial...

2017-12-07 Thread bowenli86
Github user bowenli86 commented on the issue: https://github.com/apache/flink/pull/5069 My apologies! Silly mistake at late night when the brain is quite inactive I forgot to check the files history, and thought this PR (which shows as latest commit) added those examples...

[GitHub] flink pull request #5127: [FLINK-7749][network] refactor the ResultPartition...

2017-12-07 Thread pnowojski
Github user pnowojski commented on a diff in the pull request: https://github.com/apache/flink/pull/5127#discussion_r155451532 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/Task.java --- @@ -187,8 +187,6 @@ private final ResultPartition[]

[jira] [Commented] (FLINK-8149) Replace usages of deprecated SerializationSchema

2017-12-07 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-8149?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16281476#comment-16281476 ] ASF GitHub Bot commented on FLINK-8149: --- Github user bowenli86 commented on the issue:

[GitHub] flink issue #5129: [Hotfix for State doc in 1.4 and 1.5] update State's doc ...

2017-12-07 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/5129 +1, LGTM. ---

[jira] [Commented] (FLINK-8216) Unify test utils in flink-connector-kinesis

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

[GitHub] flink pull request #5130: [FLINK-8216] Unify test utils in flink-connector-k...

2017-12-07 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/5130#discussion_r155451196 --- Diff: flink-connectors/flink-connector-kinesis/src/test/java/org/apache/flink/streaming/connectors/kinesis/testutils/TestUtils.java --- @@ -0,0 +1,39

<    1   2