[GitHub] apex-malhar pull request #503: APEXMALHAR-2301 Refactor Timebucketassigner

2016-11-20 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/503 APEXMALHAR-2301 Refactor Timebucketassigner Refactor timebucketassigner to add a simple timebucket assigner and change the bucket metadata datastructure from array to map to handle unbounded

[GitHub] apex-malhar pull request #495: APEXMALHAR-2271 Implement a multimap with tim...

2016-11-14 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/495 APEXMALHAR-2271 Implement a multimap with time extract from value You can merge this pull request into a Git repository by running: $ git pull https://github.com/siyuanh/apex-malhar

[GitHub] apex-malhar pull request #461: APEXMALHAR-2203 support control tuple and wat...

2016-10-21 Thread siyuanh
Github user siyuanh closed the pull request at: https://github.com/apache/apex-malhar/pull/461 --- 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] apex-malhar pull request #461: APEXMALHAR-2203 support control tuple and wat...

2016-10-21 Thread siyuanh
GitHub user siyuanh reopened a pull request: https://github.com/apache/apex-malhar/pull/461 APEXMALHAR-2203 support control tuple and watermark in high-level API You can merge this pull request into a Git repository by running: $ git pull https://github.com/siyuanh/apex

[GitHub] apex-malhar pull request #413: APEXMALHAR-2230 simplify the kafka input oper...

2016-09-15 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/413 APEXMALHAR-2230 simplify the kafka input operator test OK, this change doesn't fix the intermittent error directly, but it simplifies the test to make it easier to debug in the future You can

[GitHub] apex-malhar pull request #394: APEXMALHAR-2142 #comment Fix content in test ...

2016-08-30 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/394 APEXMALHAR-2142 #comment Fix content in test resource file FYI You can merge this pull request into a Git repository by running: $ git pull https://github.com/siyuanh/apex-malhar master

[GitHub] apex-malhar pull request #387: APEXMALHAR-2199 fix chroot path for 0.8 kafka...

2016-08-26 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/387 APEXMALHAR-2199 fix chroot path for 0.8 kafka input operator support Simplify the zookeeper url parser to use whatever user specified and support chroot path You can merge this pull request

[GitHub] apex-malhar pull request #365: APEXMALHAR-2028 #resolve #comment add stderr ...

2016-08-18 Thread siyuanh
GitHub user siyuanh reopened a pull request: https://github.com/apache/apex-malhar/pull/365 APEXMALHAR-2028 #resolve #comment add stderr for ConsoleOperator Add stderr to ConsleOutputOperator You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-malhar pull request #365: APEXMALHAR-2028 #resolve #comment add stderr ...

2016-08-18 Thread siyuanh
Github user siyuanh closed the pull request at: https://github.com/apache/apex-malhar/pull/365 --- 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] apex-malhar pull request #365: APEXMALHAR-2028 #resolve #comment add stderr ...

2016-08-15 Thread siyuanh
GitHub user siyuanh reopened a pull request: https://github.com/apache/apex-malhar/pull/365 APEXMALHAR-2028 #resolve #comment add stderr for ConsoleOperator Add stderr to ConsleOutputOperator You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-malhar pull request #365: APEXMALHAR-2028 #resolve #comment add stderr ...

2016-08-15 Thread siyuanh
Github user siyuanh closed the pull request at: https://github.com/apache/apex-malhar/pull/365 --- 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] apex-malhar pull request #339: APEXMALHAR-2142 Windowed Stream First Impleme...

2016-08-11 Thread siyuanh
GitHub user siyuanh reopened a pull request: https://github.com/apache/apex-malhar/pull/339 APEXMALHAR-2142 Windowed Stream First Implementation This is for review only, please ignore all checkstyle violations, format issue etc. For windowed operator changes, refer

[GitHub] apex-malhar pull request #339: APEXMALHAR-2142 Windowed Stream First Impleme...

2016-08-11 Thread siyuanh
Github user siyuanh closed the pull request at: https://github.com/apache/apex-malhar/pull/339 --- 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] apex-malhar pull request #365: APEXMALHAR-2028 #resolve #comment add stderr ...

2016-08-10 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/365 APEXMALHAR-2028 #resolve #comment add stderr for ConsoleOperator Add stderr to ConsleOutputOperator You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-malhar pull request #351: APEXMALHAR-2169 Fixed the issue of dynamic pa...

2016-08-07 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/351#discussion_r73811155 --- Diff: contrib/src/main/java/com/datatorrent/contrib/kafka/AbstractKafkaInputOperator.java --- @@ -834,17 +840,16 @@ private boolean

[GitHub] apex-malhar pull request #351: APEXMALHAR-2169 Fixed the issue of dynamic pa...

2016-08-05 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/351#discussion_r73647730 --- Diff: contrib/src/main/java/com/datatorrent/contrib/kafka/AbstractKafkaInputOperator.java --- @@ -188,6 +188,8 @@ @Min(1) private int

[GitHub] apex-core pull request #359: APEXCORE-494 Fixed the dynamic partition issue ...

2016-07-22 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-core/pull/359#discussion_r71914879 --- Diff: engine/src/main/java/com/datatorrent/stram/plan/physical/PhysicalPlan.java --- @@ -895,8 +895,10 @@ private void redoPartitions(PMapping

[GitHub] apex-malhar pull request #341: APEXMALHAR-2148 #resolve #comment Skip some n...

2016-07-13 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/341 APEXMALHAR-2148 #resolve #comment Skip some noisy loggings from kafka Skip some noisy loggings from kafka You can merge this pull request into a Git repository by running: $ git pull

[GitHub] apex-malhar pull request #327: Review-Only WindowedStream API

2016-07-12 Thread siyuanh
Github user siyuanh closed the pull request at: https://github.com/apache/apex-malhar/pull/327 --- 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] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output operator: 0.9....

2016-07-08 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r70109068 --- Diff: kafka/src/test/resources/log4j.properties --- @@ -47,3 +47,4 @@ log4j.logger.org.apache.apex=INFO log4j.logger.org.apacke.kafka=WARN

[GitHub] apex-malhar pull request #321: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-24 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/321#discussion_r68420530 --- Diff: kafka/src/test/java/org/apache/apex/malhar/kafka/KafkaInputOperatorTest.java --- @@ -264,49 +336,57 @@ public void testInputOperator(boolean

[GitHub] apex-malhar pull request #327: Review-Only WindowedStream API

2016-06-23 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/327 Review-Only WindowedStream API This is for review only, please ignore all checkstyle violations, format issue etc. For windowed operator changes, refer to https://github.com/apache

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-23 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68180895 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-23 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68180501 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-23 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68179972 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-22 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68179198 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-22 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68179058 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #298: [APEXMALHAR-2086] Kafka output: 0.9.1 first c...

2016-06-22 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/298#discussion_r68178863 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaSinglePortExactlyOnceOutputOperator.java --- @@ -0,0 +1,369

[GitHub] apex-malhar pull request #323: APEXMALHAR-2121: Updating KafkaInputOperator ...

2016-06-21 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/323#discussion_r67949068 --- Diff: contrib/src/main/java/com/datatorrent/contrib/kafka/AbstractKafkaInputOperator.java --- @@ -203,6 +204,14 @@ public

[GitHub] apex-malhar pull request #321: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/321#discussion_r67780118 --- Diff: kafka/pom.xml --- @@ -198,7 +198,7 @@ org.apache.kafka kafka_2.11 - 0.9.0.0 --- End diff

[GitHub] apex-malhar pull request #321: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/321#discussion_r67774397 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaConsumerWrapper.java --- @@ -129,6 +130,11 @@ public void emitImmediately(Map

[GitHub] apex-malhar pull request #321: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/321#discussion_r67774142 --- Diff: kafka/src/main/java/org/apache/apex/malhar/kafka/KafkaConsumerWrapper.java --- @@ -74,7 +75,7 @@ private static final Logger

[GitHub] apex-malhar pull request #321: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/321#discussion_r67770746 --- Diff: kafka/pom.xml --- @@ -198,7 +198,7 @@ org.apache.kafka kafka_2.11 - 0.9.0.0 --- End diff

[GitHub] apex-malhar pull request #320: APEXMALHAR-2120 #resolve #comment solve probl...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/320#discussion_r67769627 --- Diff: kafka/pom.xml --- @@ -198,7 +198,7 @@ org.apache.kafka kafka_2.11 - 0.9.0.0 + 0.9.0.1

[GitHub] apex-malhar pull request #323: APEXMALHAR-2121: Updating KafkaInputOperator ...

2016-06-20 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/323#discussion_r67717267 --- Diff: contrib/src/main/java/com/datatorrent/contrib/kafka/AbstractKafkaInputOperator.java --- @@ -203,6 +204,14 @@ public

[GitHub] apex-malhar pull request #316: High-level WindowedStream API (Review only)

2016-06-08 Thread siyuanh
GitHub user siyuanh opened a pull request: https://github.com/apache/apex-malhar/pull/316 High-level WindowedStream API (Review only) You can merge this pull request into a Git repository by running: $ git pull https://github.com/siyuanh/incubator-apex-malhar stream

[GitHub] incubator-apex-core pull request #338: APEXCORE-463 : fixed condition to all...

2016-06-01 Thread siyuanh
Github user siyuanh commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/338#discussion_r65421945 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/TypeGraph.java --- @@ -355,7 +356,8 @@ public int size() } Set