[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread subhankarb
Github user subhankarb commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68484062 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/common/container/RedisContainer.java ---

[jira] [Commented] (FLINK-3034) Redis SInk Connector

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

[jira] [Commented] (FLINK-4103) Modify CsvTableSource to implement StreamTableSource

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

[GitHub] flink pull request #2162: FLINK-4103: Modify CsvTableSource to implement Str...

2016-06-24 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/flink/pull/2162 FLINK-4103: Modify CsvTableSource to implement StreamTableSource Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.

[GitHub] flink pull request #2161: FLINK-4301: Modify CsvTableSource to implement Str...

2016-06-24 Thread smarthi
Github user smarthi closed the pull request at: https://github.com/apache/flink/pull/2161 --- 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 issue #2161: FLINK-4301: Modify CsvTableSource to implement StreamTabl...

2016-06-24 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/flink/pull/2161 Wrong Jira# closing 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 feature enabled and

[GitHub] flink pull request #2161: FLINK-4301: Modify CsvTableSource to implement Str...

2016-06-24 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/flink/pull/2161 FLINK-4301: Modify CsvTableSource to implement StreamTableSource Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.

[jira] [Commented] (FLINK-3580) Reintroduce Date/Time and implement scalar functions for it

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

[GitHub] flink issue #2150: [FLINK-3580] [table] Reintroduce Date/Time and implement ...

2016-06-24 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/2150 Hi @twalthr, thanks for the PR. I skimmed the changes and everything looked fine. I have two suggestions though. 1. Can we add a casts from `Long` to `Timestamp` and back? 2. Can you

[jira] [Commented] (FLINK-3580) Reintroduce Date/Time and implement scalar functions for it

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

[GitHub] flink pull request #2150: [FLINK-3580] [table] Reintroduce Date/Time and imp...

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2150#discussion_r68476229 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/typeutils/TypeCoercion.scala --- @@ -85,6 +85,9 @@ object TypeCoercion {

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink issue #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on the issue: https://github.com/apache/flink/pull/2159 Hi @wuchong, thanks for the PR. I think it is better to use a CoGroup instead of a Join for `INTERSECT` (even though, I said "Join" in the original JIRA). For `INTERSECT ALL`, CoGroup is required.

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68474570 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/codegen/CodeGenerator.scala --- @@ -260,10 +260,14 @@ class CodeGenerator(

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68474420 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/api/scala/batch/sql/IntersectITCase.scala --- @@ -0,0 +1,108 @@ +/* + *

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68474131 --- Diff: flink-libraries/flink-table/src/test/java/org/apache/flink/api/java/batch/table/IntersectITCase.java --- @@ -0,0 +1,101 @@ +/* + *

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68473826 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/table.scala --- @@ -446,6 +446,31 @@ class Table( } /**

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68473636 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/table.scala --- @@ -446,6 +446,31 @@ class Table( } /**

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68473250 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/dataset/DataSetIntersect.scala --- @@ -0,0 +1,153 @@ +/*

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68473118 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/dataset/DataSetIntersect.scala --- @@ -0,0 +1,153 @@ +/*

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68473031 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/dataset/DataSetIntersect.scala --- @@ -0,0 +1,153 @@ +/*

[jira] [Commented] (FLINK-3942) Add support for INTERSECT

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

[GitHub] flink pull request #2159: [FLINK-3942] [tableAPI] Add support for INTERSECT

2016-06-24 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2159#discussion_r68471662 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/table/plan/nodes/dataset/DataSetIntersect.scala --- @@ -0,0 +1,153 @@ +/*

[jira] [Commented] (FLINK-4053) Return value from Connection should be checked against null

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

[GitHub] flink issue #2128: [FLINK-4053] Return value from Connection should be check...

2016-06-24 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2128 @zentol I've fixed my PR according to your comments. --- 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-3898) Adamic-Adar Similarity

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

[GitHub] flink pull request #2160: [FLINK-3898] [gelly] Adamic-Adar Similarity

2016-06-24 Thread greghogan
GitHub user greghogan opened a pull request: https://github.com/apache/flink/pull/2160 [FLINK-3898] [gelly] Adamic-Adar Similarity You can merge this pull request into a Git repository by running: $ git pull https://github.com/greghogan/flink 3898_adamic_adar_similarity

[jira] [Commented] (FLINK-3034) Redis SInk Connector

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

[jira] [Closed] (FLINK-3907) Directed Clustering Coefficient

2016-06-24 Thread Greg Hogan (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-3907?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Greg Hogan closed FLINK-3907. - Resolution: Fixed Implemented in d34bdaf7f1de4dd4f742867e6b9e879c8b631765 > Directed Clustering

[GitHub] flink issue #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread mjsax
Github user mjsax commented on the issue: https://github.com/apache/flink/pull/1813 @rmetzger Thanks :) I did not test with a Redis cluster or similar. --- 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

[GitHub] flink issue #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/1813 @mjsax, did you test the code with a redis installation / on a cluster? (I'm not expecting those tests from a PR review, but I would not do it again if you already did ... and I'm not doing it

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68455277 --- Diff: flink-streaming-connectors/flink-connector-redis/src/test/java/org/apache/flink/streaming/connectors/redis/RedisSentinelClusterTest.java --- @@

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68455334 --- Diff: flink-streaming-connectors/flink-connector-redis/src/test/java/org/apache/flink/streaming/connectors/redis/RedisSentinelClusterTest.java --- @@

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68455103 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/common/container/RedisContainer.java ---

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68454693 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/common/container/RedisContainer.java ---

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68454559 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/common/container/RedisClusterContainer.java

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68454370 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/RedisSink.java --- @@ -0,0 +1,232 @@

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68453882 --- Diff: flink-streaming-connectors/flink-connector-redis/pom.xml --- @@ -0,0 +1,92 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68453630 --- Diff: flink-streaming-connectors/flink-connector-redis/pom.xml --- @@ -0,0 +1,92 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] flink issue #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread subhankarb
Github user subhankarb commented on the issue: https://github.com/apache/flink/pull/1813 done. --- 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

[GitHub] flink pull request #2079: [FLINK-3907] [gelly] Directed Clustering Coefficie...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2079 --- 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 #2137: [FLINK-4041] reduce log level for JobManager->Reso...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2137 --- 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 #2125: [FLINK-3864] Yarn tests don't check for prohibited...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2125 --- 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 #2138: [FLINK-3757] clarify JavaDoc for addAccumulator me...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2138 --- 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 #2139: [FLINK-3838] Upgrade commons-cli to fix parsing of...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/2139 --- 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-4098) Iteration support in Python API

2016-06-24 Thread Chesnay Schepler (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15348529#comment-15348529 ] Chesnay Schepler commented on FLINK-4098: - huh. As far as i can tell, every call to next() in

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink issue #2131: [FLINK-3231][streaming-connectors] FlinkKinesisConsumer r...

2016-06-24 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2131 Okay, sounds good. --- 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,

[GitHub] flink issue #2131: [FLINK-3231][streaming-connectors] FlinkKinesisConsumer r...

2016-06-24 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2131 @rmetzger Thank you! No problem, I can wait and address the current comments meanwhile. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[jira] [Commented] (FLINK-4018) Configurable idle time between getRecords requests to Kinesis shards

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

[GitHub] flink issue #2131: [FLINK-3231][streaming-connectors] FlinkKinesisConsumer r...

2016-06-24 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2131 I'm not done with the review yet. I hope I find some time over the weekend to continue with it. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] flink issue #2071: [FLINK-4018][streaming-connectors] Configurable idle time...

2016-06-24 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/2071 Yes, I would suggest to do this PR after the big one. --- 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-4018) Configurable idle time between getRecords requests to Kinesis shards

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

[GitHub] flink issue #2071: [FLINK-4018][streaming-connectors] Configurable idle time...

2016-06-24 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/2071 Thank you for reviewing this @rmetzger. Should I address your comment after the rework in https://github.com/apache/flink/pull/2131 is merged, and rebase this PR on that? --- If your project is

[jira] [Commented] (FLINK-4018) Configurable idle time between getRecords requests to Kinesis shards

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

[GitHub] flink pull request #2071: [FLINK-4018][streaming-connectors] Configurable id...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2071#discussion_r68420387 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/internals/ShardConsumerThread.java ---

[jira] [Comment Edited] (FLINK-4098) Iteration support in Python API

2016-06-24 Thread Geoffrey Mon (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15347577#comment-15347577 ] Geoffrey Mon edited comment on FLINK-4098 at 6/24/16 3:34 PM: -- Thanks for the

[jira] [Commented] (FLINK-3034) Redis SInk Connector

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

[GitHub] flink issue #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread mjsax
Github user mjsax commented on the issue: https://github.com/apache/flink/pull/1813 Please address last comment. Otherwise, LGTM. --- 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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68414043 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/proxy/KinesisProxy.java --- @@

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68413466 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/proxy/KinesisProxy.java --- @@

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[jira] [Commented] (FLINK-3034) Redis SInk Connector

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

[GitHub] flink pull request #1813: [FLINK-3034] Redis Sink Connector

2016-06-24 Thread mjsax
Github user mjsax commented on a diff in the pull request: https://github.com/apache/flink/pull/1813#discussion_r68412669 --- Diff: flink-streaming-connectors/flink-connector-redis/src/main/java/org/apache/flink/streaming/connectors/redis/common/container/RedisClusterContainer.java

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68412025 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/proxy/KinesisProxy.java --- @@

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68411242 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/model/KinesisStreamShard.java ---

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68410216 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/model/KinesisStreamShard.java ---

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68409932 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/model/KinesisStreamShard.java ---

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68409807 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/model/KinesisStreamShard.java ---

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68409331 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/proxy/KinesisProxy.java --- @@

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread tzulitai
Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68408986 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/internals/KinesisDataFetcher.java ---

[jira] [Created] (FLINK-4117) Wait for CuratorFramework connection to be established

2016-06-24 Thread Greg Hogan (JIRA)
Greg Hogan created FLINK-4117: - Summary: Wait for CuratorFramework connection to be established Key: FLINK-4117 URL: https://issues.apache.org/jira/browse/FLINK-4117 Project: Flink Issue Type:

[jira] [Commented] (FLINK-4053) Return value from Connection should be checked against null

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

[GitHub] flink pull request #2128: [FLINK-4053] Return value from Connection should b...

2016-06-24 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/2128#discussion_r68408173 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/test/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQSinkTest.java --- @@

[jira] [Commented] (FLINK-4113) Always copy first value in ChainedAllReduceDriver

2016-06-24 Thread Greg Hogan (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-4113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15348356#comment-15348356 ] Greg Hogan commented on FLINK-4113: --- I experienced the problem when verifying the

[jira] [Commented] (FLINK-4053) Return value from Connection should be checked against null

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

[GitHub] flink issue #2128: [FLINK-4053] Return value from Connection should be check...

2016-06-24 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2128 I wanted to avoid this bit of duplication, but I have no strong preferences for that. Will remove this static method. --- If your project is set up for it, you can reply to this email and have

[GitHub] flink pull request #2128: [FLINK-4053] Return value from Connection should b...

2016-06-24 Thread zentol
Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/2128#discussion_r68406896 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/test/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQSinkTest.java --- @@

[jira] [Commented] (FLINK-4053) Return value from Connection should be checked against null

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

[jira] [Commented] (FLINK-3231) Handle Kinesis-side resharding in Kinesis streaming consumer

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

[jira] [Commented] (FLINK-4053) Return value from Connection should be checked against null

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

[GitHub] flink pull request #2131: [FLINK-3231][streaming-connectors] FlinkKinesisCon...

2016-06-24 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2131#discussion_r68406625 --- Diff: flink-streaming-connectors/flink-connector-kinesis/src/main/java/org/apache/flink/streaming/connectors/kinesis/model/KinesisStreamShard.java ---

  1   2   >