[GitHub] incubator-apex-core pull request: APEXCORE-466 #resolve improved l...

2016-05-20 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/339 APEXCORE-466 #resolve improved logging from the *Agent.java files @PramodSSImmaneni please merge You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-apex-core pull request: APEXCORE-463 : fixed condition t...

2016-05-19 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/338#issuecomment-220407589 @siyuanh should review 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

[GitHub] incubator-apex-core pull request: APEXCORE-462 Cache the invalid w...

2016-05-18 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/336 APEXCORE-462 Cache the invalid web services info so that yarn requests won't be repeated unnecessarily @PramodSSImmaneni please review You can merge this pull request into

[GitHub] incubator-apex-malhar pull request: Added travis define so that we...

2016-05-17 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-malhar/pull/280 Added travis define so that we can skip certain unit tests in travis if we want to @tweise please merge You can merge this pull request into a Git repository by running

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624566 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/ApexStream.java --- @@ -0,0 +1,261 @@ +/** + * Licensed to the

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624687 --- Diff: library/src/main/java/com/datatorrent/lib/algo/UniqueCounter.java --- @@ -78,10 +78,10 @@ public void process(K tuple

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624506 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/impl/ApexStreamImpl.java --- @@ -0,0 +1,485 @@ +/** + * Licensed

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624391 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/operator/FunctionOperator.java --- @@ -0,0 +1,361

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624361 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/operator/FunctionOperator.java --- @@ -0,0 +1,361

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-17 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63624253 --- Diff: stream/src/test/java/org/apache/apex/malhar/stream/sample/MyStreamTest.java --- @@ -0,0 +1,90 @@ +/** + * Licensed to

[GitHub] incubator-apex-core pull request: APEXCORE-458 Using the yarn conf...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/334#discussion_r63443931 --- Diff: engine/src/main/java/com/datatorrent/stram/client/StramClientUtils.java --- @@ -739,13 +745,21 @@ public static InetSocketAddress

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443653 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443609 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443528 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443468 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443413 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443369 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443303 --- Diff: docs/security.md --- @@ -142,7 +168,15 @@ When operators are running there will be effective processing rate differences b

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443227 --- Diff: docs/security.md --- @@ -74,6 +74,32 @@ The snippet below shows the how the credentials can be specified in the configur

[GitHub] incubator-apex-core pull request: APEXCORE-457 Added web service s...

2016-05-16 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/333#discussion_r63443019 --- Diff: docs/security.md --- @@ -74,6 +74,32 @@ The snippet below shows the how the credentials can be specified in the configur

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63109851 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/impl/ApexStreamImpl.java --- @@ -0,0 +1,457 @@ +/** + * Licensed

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63109747 --- Diff: stream/src/main/java/org/apache/apex/malhar/stream/api/impl/StreamFactory.java --- @@ -0,0 +1,81 @@ +/** + * Licensed

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63100920 --- Diff: pom.xml --- @@ -197,6 +197,7 @@ library contrib +stream --- End diff -- should

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63100793 --- Diff: demos/wordcount/src/test/java/com/datatorrent/demos/wordcount/LocalTestWithoutStreamApplication.java --- @@ -0,0 +1,48

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63100731 --- Diff: demos/wordcount/src/test/java/com/datatorrent/demos/wordcount/ApplicationWithStreamAPITest.java --- @@ -0,0 +1,49

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2006 Stream API

2016-05-12 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/261#discussion_r63100120 --- Diff: contrib/src/main/java/com/datatorrent/contrib/kafka/SimpleKafkaConsumer.java --- @@ -277,7 +277,7 @@ public SimpleKafkaConsumer

[GitHub] incubator-apex-core pull request: APEXCORE-453 decreased the heart...

2016-05-10 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/329#issuecomment-218303369 @tweise 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

[GitHub] incubator-apex-core pull request: APEXCORE-453 decreased the heart...

2016-05-10 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/329#issuecomment-218274161 @sandeepdeshmukh thanks for verifying this fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-apex-core pull request: Release 3.4.0 related changes (m...

2016-05-10 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/331#issuecomment-218254758 @PramodSSImmaneni fyi, I went ahead and merged this since it's rather straight forward. --- If your project is set up for it, you can reply to

[GitHub] incubator-apex-core pull request: Release 3.4

2016-05-10 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/330#issuecomment-218242413 Everything looks good, but a lot of these changes (except the parts with 3.4.1-SNAPSHOT) need to be done to the master branch as well. --- If your

[GitHub] incubator-apex-core pull request: APEXCORE-451 : get-app-package-o...

2016-05-10 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/326#issuecomment-218234459 looks fine --- 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

[GitHub] incubator-apex-core pull request: APEXCORE-453 decreased the heart...

2016-05-09 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/329 APEXCORE-453 decreased the heartbeat interval in the DelayOperator unit test because otherwise the committed window call may occur much later than expected, hence missing the failure

[GitHub] incubator-apex-core pull request: APEXCORE-451 : get-app-package-o...

2016-05-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/326#discussion_r62241801 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/OperatorDiscoverer.java --- @@ -526,6 +544,17 @@ public JSONObject

[GitHub] incubator-apex-core pull request: APEXCORE-451 : get-app-package-o...

2016-05-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/326#discussion_r62241719 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/TypeGraph.java --- @@ -348,13 +348,13 @@ public int size() public

[GitHub] incubator-apex-site pull request: Adding announcements and meetups...

2016-05-04 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-site/pull/36#issuecomment-217038509 I'm +1 with the change. --- 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 pr

[GitHub] incubator-apex-core pull request: APEXCORE-427 use system property...

2016-05-04 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/325 APEXCORE-427 use system property instead of environment variable. Apparently environment variable don't propagate to the unit test jvm @tweise please review You can merge this

[GitHub] incubator-apex-malhar pull request: REVIEW ONLY: APEXCORE-442 Remo...

2016-05-04 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-malhar/pull/256 --- 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

[GitHub] incubator-apex-core pull request: REVIEW ONLY: APEXCORE-442 Remove...

2016-05-04 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/317 REVIEW ONLY: APEXCORE-442 Removed incubating For review only. Do not merge until our mailing list address and our repository names are updated You can merge this pull request

[GitHub] incubator-apex-core pull request: REVIEW ONLY: APEXCORE-442 Remove...

2016-05-04 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/317 --- 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

[GitHub] incubator-apex-core pull request: APEXCORE-427 disable tests that ...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/323#issuecomment-216700890 @tweise please review --- 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] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-05-03 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-core pull request: APEXCORE-427 disable tests that ...

2016-05-03 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/323 APEXCORE-427 disable tests that produce false negatives in travis You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: APEXCORE-442 Removed incubator w...

2016-05-03 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/322 APEXCORE-442 Removed incubator wording except mailing lists and repository names @tweise please review You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61951218 --- Diff: engine/src/main/java/com/datatorrent/stram/cli/ApexCli.java --- @@ -3366,6 +3370,28 @@ public void execute(String[] args

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61926171 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/StramWebServices.java --- @@ -122,7 +124,11 @@ public static final String

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61925721 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61925623 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-03 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61925340 --- Diff: engine/src/main/java/com/datatorrent/stram/cli/ApexCli.java --- @@ -3366,6 +3370,28 @@ public void execute(String[] args

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/321#issuecomment-216412358 @PramodSSImmaneni Any thread that has a stack trace that is running from the StreamingContainer class is the operator main thread, but I don't thi

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61824191 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61824215 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61823925 --- Diff: engine/src/main/java/com/datatorrent/stram/cli/ApexCli.java --- @@ -3366,6 +3370,28 @@ public void execute(String[] args

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61823831 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61823783 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61823801 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61823741 --- Diff: engine/src/main/java/com/datatorrent/stram/StramUtils.java --- @@ -81,4 +89,48 @@ } } + public static

[GitHub] incubator-apex-core pull request: APEXCORE-107 Support for adding ...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/313#issuecomment-216389226 I think in the field of streaming technology, all of these different nodes in the DAG share the same concept of an operator. So, instead of introducing

[GitHub] incubator-apex-core pull request: APEXCORE-107 Support for adding ...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/313#discussion_r61787475 --- Diff: engine/src/main/java/com/datatorrent/stram/plan/logical/LogicalPlanConfiguration.java --- @@ -2187,6 +2190,27 @@ public void

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/321#issuecomment-216319603 We will also need cli support. ``` get-container-stacks ``` --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61780111 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/StreamingContainer.java --- @@ -690,8 +693,19 @@ public void heartbeatLoop

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779943 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779550 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/StramWebServices.java --- @@ -492,6 +494,42 @@ public JSONObject getContainer

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779390 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/StramWebServices.java --- @@ -492,6 +494,42 @@ public JSONObject getContainer

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779278 --- Diff: engine/src/main/java/com/datatorrent/stram/webapp/StramWebServices.java --- @@ -492,6 +494,42 @@ public JSONObject getContainer

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779139 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61779008 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61777915 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r6191 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-core pull request: APEXCORE-330 : Stacktrace of the...

2016-05-02 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/321#discussion_r61777550 --- Diff: engine/src/main/java/com/datatorrent/stram/util/StackTrace.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2067 Necessary chan...

2016-04-27 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-malhar/pull/258 APEXMALHAR-2067 Necessary changes to work with Apex Core 3.4.0 due to ning.com:async-http-client and OperatorContext @tweise please review You can merge this pull request into a

[GitHub] incubator-apex-core pull request: APEXCORE-446 Added shaded source...

2016-04-26 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/318 APEXCORE-446 Added shaded source jar for shaded-ning19 @tweise please review You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-apex-site pull request: REVIEW ONLY: APEXCORE-442 Remove...

2016-04-25 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-site/pull/35 REVIEW ONLY: APEXCORE-442 Removed incubating For review only. Do not merge until our mailing list address and our repository names are updated You can merge this pull request into a

[GitHub] incubator-apex-malhar pull request: REVIEW ONLY: APEXCORE-442 Remo...

2016-04-25 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-malhar/pull/256 REVIEW ONLY: APEXCORE-442 Removed incubating For review only. Do not merge until our mailing list address and our repository names are updated You can merge this pull request

[GitHub] incubator-apex-core pull request: REVIEW ONLY: APEXCORE-442 Remove...

2016-04-25 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/317 REVIEW ONLY: APEXCORE-442 Removed incubating For review only. Do not merge until our mailing list address and our repository names are updated You can merge this pull request into

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-1965 WAL Utility [F...

2016-04-20 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/242#discussion_r60456872 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/wal/FileSystemWAL.java --- @@ -0,0 +1,598 @@ +/** + * Licensed to the

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-1965 WAL Utility [F...

2016-04-19 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-malhar/pull/242#discussion_r60334158 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/wal/FileSystemWAL.java --- @@ -0,0 +1,598 @@ +/** + * Licensed to the

[GitHub] incubator-apex-core pull request: APEXCORE-434 Workaround for Jers...

2016-04-18 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/310#issuecomment-211511288 @PramodSSImmaneni do we want to also cherry pick this to the release-3.3 branch? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-15 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-15 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-malhar pull request: APEXMALHAR-2054 Made App Data ...

2016-04-14 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-malhar/pull/241 APEXMALHAR-2054 Made App Data Pi Demo use an embedded Query operator @ilooner please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/308 --- 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

[GitHub] incubator-apex-core pull request: DO NOT MERGE: diagnosing the tes...

2016-04-12 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/308 DO NOT MERGE: diagnosing the testAppDataPush Travis problem You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] incubator-apex-core pull request: APEXCORE-340 Naming change of ap...

2016-04-12 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/303#issuecomment-209087975 @chinmaykolhatkar The PR looks good to me. I'll let this sit around for a couple days so that other people can chime in before I merge it. --- If

[GitHub] incubator-apex-core pull request: APEXCORE-427 Made pubsub connect...

2016-04-11 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/302#issuecomment-208626432 @vrozov please review --- 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] incubator-apex-core pull request: APEXCORE-427 Made pubsub connect...

2016-04-11 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/302 APEXCORE-427 Made pubsub connect timeout configurable You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74/incubator-apex

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-08 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/294#issuecomment-207630632 @tweise explanation is added in the comment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-07 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/294 --- 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

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-07 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/294 APEXCORE-411 support restarting app without app package or jar files @tweise please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-apex-core pull request: APEXCORE-423 style fix for apex-...

2016-04-07 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/incubator-apex-core/pull/299 APEXCORE-423 style fix for apex-core engine @chandnisingh please merge. thanks. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-05 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/incubator-apex-core/pull/294 --- 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

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-05 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/incubator-apex-core/pull/294 APEXCORE-411 support restarting app without app package or jar files @tweise please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-apex-core pull request: APEXCORE-411 support restarting ...

2016-04-05 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/incubator-apex-core/pull/294#discussion_r58595340 --- Diff: engine/src/main/java/com/datatorrent/stram/client/StramAppLauncher.java --- @@ -210,14 +239,67 @@ public StramAppLauncher

[GitHub] incubator-apex-core pull request: APEXCORE-415 #resolve #comment F...

2016-04-04 Thread davidyan74
Github user davidyan74 commented on the pull request: https://github.com/apache/incubator-apex-core/pull/292#issuecomment-205560349 @ilooner OK --- 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

  1   2   3   4   >