[GitHub] flink pull request #4422: [FLINK-7299][AVRO] Write GenericRecord using AvroO...

2017-09-01 Thread soniclavier
Github user soniclavier closed the pull request at: https://github.com/apache/flink/pull/4422 --- 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

[GitHub] flink issue #4422: [FLINK-7299][AVRO] Write GenericRecord using AvroOutputFo...

2017-08-31 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/4422 @aljoscha Thanks for taking a look. I have modified the test. --- 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 issue #4422: [FLINK-7299][AVRO] Write GenericRecord using AvroOutputFo...

2017-07-31 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/4422 Link to passed Travis build : https://travis-ci.org/soniclavier/flink/builds/259243850 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] flink pull request #4422: [FLINK-7299][AVRO] Write GenericRecord using AvroO...

2017-07-30 Thread soniclavier
GitHub user soniclavier opened a pull request: https://github.com/apache/flink/pull/4422 [FLINK-7299][AVRO] Write GenericRecord using AvroOutputFormat ## What is the purpose of the change Allow writing Avro GenericRecord using AvroOutputFormat. ## Brief change log

[GitHub] flink pull request #3763: [FLINK-6372][scripts] Fix change scala version of ...

2017-05-03 Thread soniclavier
Github user soniclavier closed the pull request at: https://github.com/apache/flink/pull/3763 --- 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

[GitHub] flink pull request #3763: [FLINK-6372][scripts] Fix change scala version of ...

2017-04-24 Thread soniclavier
GitHub user soniclavier opened a pull request: https://github.com/apache/flink/pull/3763 [FLINK-6372][scripts] Fix change scala version of flink-gelly-examples - changed change-scala-version.sh You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] flink issue #2736: [FLINK-4174] Enhance evictor functionality

2016-11-15 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2736 I don't think I can edit a closed issue, could you please make the edit. --- 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

[GitHub] flink issue #2736: [FLINK-4174] Enhance evictor functionality

2016-11-15 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2736 Thank you for your guidance @aljoscha 😄. Could you please tell me what Fix version I should keep for these JIRAs --- If your project is set up for it, you can reply to this email and have

[GitHub] flink pull request #2736: [FLINK-4174] Enhance evictor functionality

2016-11-15 Thread soniclavier
Github user soniclavier closed the pull request at: https://github.com/apache/flink/pull/2736 --- 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

[GitHub] flink issue #2736: [FLINK-4174] Enhance evictor functionality

2016-11-14 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2736 @aljoscha made the changes as per your comments, could you please review it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] flink pull request #2355: [FLINK-4282]Add Offset Parameter to WindowAssigner...

2016-11-04 Thread soniclavier
Github user soniclavier commented on a diff in the pull request: https://github.com/apache/flink/pull/2355#discussion_r86656836 --- Diff: flink-streaming-java/src/main/java/org/apache/flink/streaming/api/windowing/windows/TimeWindow.java --- @@ -236,6 +236,6 @@ public int compare

[GitHub] flink issue #2736: [FLINK-4174] Enhance evictor functionality

2016-11-04 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2736 @aljoscha I have made the changes, could you please review it. --- 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 pull request #2736: [FLINK-4174] Enhance evictor functionality

2016-11-03 Thread soniclavier
Github user soniclavier commented on a diff in the pull request: https://github.com/apache/flink/pull/2736#discussion_r86462637 --- Diff: flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/operators/windowing/EvictingWindowOperator.java --- @@ -273,32 +284,93

[GitHub] flink pull request #2736: [FLINK-4174] Enhance evictor functionality

2016-10-31 Thread soniclavier
GitHub user soniclavier opened a pull request: https://github.com/apache/flink/pull/2736 [FLINK-4174] Enhance evictor functionality The PR implements [FLINK-4174](https://issues.apache.org/jira/browse/FLINK-4174) Enhance window evictor as proposed in [FLIP-4](https

[GitHub] flink issue #2051: [FLINK-3779] Add support for queryable state

2016-07-21 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2051 One more question, is it possible to configure the JobManager Actor path that the client connects to, it looks like it default to `akka://flink/user/jobmanager`. In that way I can create

[GitHub] flink issue #2051: [FLINK-3779] Add support for queryable state

2016-07-21 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2051 Sorry, the compilation error was because the Tuple2 was scala.Tuple2 not flink Tuple2. Changing to `org.apache.flink.api.java.tuple.Tuple2` fixed the issue. --- If your project is set up

[GitHub] flink issue #2051: [FLINK-3779] Add support for queryable state

2016-07-21 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2051 Thanks Ufuk & Stephen for the reply, I tried the serializers suggested by you ``` val typeHint = new TypeHint[Tuple2[Long,String]](){} val serializer = TypeInformatio

[GitHub] flink issue #2051: [FLINK-3779] Add support for queryable state

2016-07-21 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2051 Never mind, I was hitting with wrong key, it works now! Cheers. --- 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 #2051: [FLINK-3779] Add support for queryable state

2016-07-21 Thread soniclavier
Github user soniclavier commented on the issue: https://github.com/apache/flink/pull/2051 Hi, Continuing the discussion from the mailing list, I was able to go past the NettyConfig problem once I ran Flink in cluster mode ( I would still like to know if there is a way to run