[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-06-01 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65328505 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread subhankarb
Github user subhankarb commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65303646 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/2054 Thank you for your contribution. There are still some open issues that need addressing before we can merge this. --- If your project is set up for it, you can reply to this email and

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65253337 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/RMQSink.java --- @@ -35,24

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65253122 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/RMQSink.java --- @@ -35,24

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65253162 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/RMQSink.java --- @@ -35,24

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65253016 --- Diff: docs/apis/streaming/connectors/rabbitmq.md --- @@ -71,23 +71,25 @@ Example: {% highlight java %} +RMQConnectionConfig

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65252614 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65252489 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65251121 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65250839 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---

[GitHub] flink pull request: [FLINK-3763] RabbitMQ Source/Sink standardize connection...

2016-05-31 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/2054#discussion_r65250708 --- Diff: flink-streaming-connectors/flink-connector-rabbitmq/src/main/java/org/apache/flink/streaming/connectors/rabbitmq/common/RMQConnectionConfig.java ---