[GitHub] apex-site pull request #76: Changed default mail archive to mailing list fro...

2017-07-08 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-site/pull/76 Changed default mail archive to mailing list from mod_mbox @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex

[GitHub] apex-core pull request #553: APEXCORE-753 Add mailing lists, SCM and issue t...

2017-07-05 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/553 APEXCORE-753 Add mailing lists, SCM and issue tracking information to POM @tweise Please review and merge You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #550: APEXCORE-747 Provide additional ToStringStyle o...

2017-06-25 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/550 APEXCORE-747 Provide additional ToStringStyle options @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #549: APEXCORE-749 Fix README.md formatting

2017-06-22 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/549 APEXCORE-749 Fix README.md formatting @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core APEXCORE-749

[GitHub] apex-core pull request #548: APEXCORE-748 Upgrade netlet dependency to 1.3.2

2017-06-21 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/548 APEXCORE-748 Upgrade netlet dependency to 1.3.2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core APEXCORE-748 Alternatively

[GitHub] apex-core pull request #542: APEXCORE-745 Buffer server may stop processing ...

2017-06-13 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/542 APEXCORE-745 Buffer server may stop processing tuples when backpressure is enabled @PramodSSImmaneni, @sandeshh Please review You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #542: APEXCORE-745 Buffer server may stop processing ...

2017-06-13 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/542 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/533 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/533 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/533 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/533 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/533 --- 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] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-06-10 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #542: APEXCORE-745 Buffer server may stop processing ...

2017-06-08 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/542 APEXCORE-745 Buffer server may stop processing tuples when backpressure is enabled @PramodSSImmaneni, @sandeshh Please review You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #537: APEXCORE-741 Upgrade netlet dependency to 1.3.1

2017-06-05 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/537 APEXCORE-741 Upgrade netlet dependency to 1.3.1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core APEXCORE-741 Alternatively

[GitHub] apex-core pull request #533: APEXCORE-735 Upgrade maven-dependency-plugin

2017-05-30 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/533 APEXCORE-735 Upgrade maven-dependency-plugin @tweise @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core pull request #532: APEXCORE-734 StramLocalCluster may not terminat...

2017-05-30 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/532 APEXCORE-734 StramLocalCluster may not terminate properly @PramodSSImmaneni @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #524: APEXCORE-717 Remove unnecessary archetypeVersio...

2017-05-07 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/524 APEXCORE-717 Remove unnecessary archetypeVersion property @tweise Please see. You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core

[GitHub] apex-core pull request #521: APEXCORE-715 Remove unnecessary @Evolving annot...

2017-04-27 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/521 APEXCORE-715 Remove unnecessary @Evolving annotation in engine You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core APEXCORE-715

[GitHub] apex-core pull request #516: APEXCORE-703 Window processing timeout for fini...

2017-04-16 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/516 APEXCORE-703 Window processing timeout for finished/undeployed container. During an operator shutdown, mark it as INACTIVE to exclude it from the blocked operators check. @tweise Please

[GitHub] apex-core pull request #506: APEXCORE-695 Remove unnecessary interface modif...

2017-04-09 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/506 APEXCORE-695 Remove unnecessary interface modifiers @tweise Please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core

[GitHub] apex-core pull request #505: APEXCORE-691 Use type inference for generic ins...

2017-04-07 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/505 APEXCORE-691 Use type inference for generic instance creation @tweise Please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex

[GitHub] apex-core pull request #502: APEXCORE-686 AppPackage fails when .apa contain...

2017-03-30 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/502 APEXCORE-686 AppPackage fails when .apa contains zero length stored entry @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #496: APEXCORE-683 Apex client should support applica...

2017-03-25 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/496 APEXCORE-683 Apex client should support application packages on HDFS You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core

[GitHub] apex-core pull request #492: APEXCORE-677 Avoid starting StramLocalCluster i...

2017-03-21 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/492 APEXCORE-677 Avoid starting StramLocalCluster in StreamingContainerManagerTest.testAppDataSources @tweise Please merge You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #481: APEXCORE-656 Upgrade org.apache.httpcomponents....

2017-03-21 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/481 --- 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] apex-malhar pull request #580: APEXMALHAR-2306 Tests should allow for additi...

2017-03-13 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-malhar/pull/580 APEXMALHAR-2306 Tests should allow for additions to OperatorContext interface @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #481: APEXCORE-656 Upgrade org.apache.httpcomponents....

2017-02-28 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/481 APEXCORE-656 Upgrade org.apache.httpcomponents.httpclient. @PramodSSImmaneni please review and merge You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #477: APEXCORE-648 Unnecessary byte array copy in Def...

2017-02-22 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/477 APEXCORE-648 Unnecessary byte array copy in DefaultStatefulStreamCodec.toDataStatePair() @sandeshh Please review and merge You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #472: APEXCORE-645 StramLocalCluster does not wait fo...

2017-02-13 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/472 APEXCORE-645 StramLocalCluster does not wait for master thread termination @tweise or @PramodSSImmaneni please merge You can merge this pull request into a Git repository by running: $ git

[GitHub] apex-core pull request #471: APEXCORE-597 BufferServer needs to shutdown all...

2017-02-12 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/471 APEXCORE-597 BufferServer needs to shutdown all created execution services @PramodSSImmaneni, @tweise, @tushargosavi or @sandeshh Please review You can merge this pull request into a Git

[GitHub] apex-core pull request #470: APEXCORE-641 Subscribers/DataListeners may not ...

2017-02-10 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/470 APEXCORE-641 Subscribers/DataListeners may not be scheduled to execute even when they have data to process @PramodSSImmaneni, @tweise, @sandeshh Please review. You can merge this pull request

[GitHub] apex-core pull request #461: APEXCORE-504 - Possible race condition in Strea...

2017-01-30 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/461 APEXCORE-504 - Possible race condition in StreamingContainerAgent.getStreamCodec() @PramodSSImmaneni or @tweise Please review You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #451: APEXCORE-617 InputNodeTest intermittently fails...

2017-01-21 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/451 APEXCORE-617 InputNodeTest intermittently fails with ConcurrentModificationException @ilooner or @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #449: Set right margin to 120. Do not wrap long lines...

2017-01-20 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/449 Set right margin to 120. Do not wrap long lines. @davidyan74 Please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core

[GitHub] apex-core pull request #448: APEXCORE-608 Streaming Containers use stale RPC...

2017-01-19 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/448 APEXCORE-608 Streaming Containers use stale RPC proxy after connection is closed @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #444: APEXCORE-605 Suppress bootstrap compiler warnin...

2017-01-09 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/444 APEXCORE-605 Suppress bootstrap compiler warning @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core

[GitHub] apex-core pull request #438: APEXCORE-504 - Possible race condition in Strea...

2016-12-30 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/438 APEXCORE-504 - Possible race condition in StreamingContainerAgent.getStreamCodec() @PramodSSImmaneni, @tweise Please review and merge. You can merge this pull request into a Git repository by

[GitHub] apex-core pull request #436: APEXCORE-456 - Explicitly limit Server.Subscrib...

2016-12-19 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/436 APEXCORE-456 - Explicitly limit Server.Subscriber to one way communication @PramodSSImmaneni Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #434: APEXCORE-591 - SubscribeRequestTuple has wrong ...

2016-12-16 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/434 APEXCORE-591 - SubscribeRequestTuple has wrong buffer size when mask is zero @tushargosavi Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #431: Add Apache 2.0 license to META-INF/services fil...

2016-12-12 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/431 Add Apache 2.0 license to META-INF/services files @PramodSSImmaneni Please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex

[GitHub] apex-core pull request #428: APEXCORE-583 - Buffer Server LogicalNode should...

2016-12-02 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/428 APEXCORE-583 - Buffer Server LogicalNode should not be reused by Subscribers @PramodSSImmaneni, @tweise, @243826 Please review. You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #420: APEXCORE-456 - Explicitly limit Server.Subscrib...

2016-11-20 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/420 APEXCORE-456 - Explicitly limit Server.Subscriber to one way communication @PramodSSImmaneni Please review only. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] apex-core pull request #419: APEXCORE-251 - Suppress "Journal output stream ...

2016-11-15 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/419 APEXCORE-251 - Suppress "Journal output stream is null..." warning in unit tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/v

[GitHub] apex-core pull request #418: APEXCORE-557 - Upgrade netlet dependency to 1.3...

2016-11-14 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/418 APEXCORE-557 - Upgrade netlet dependency to 1.3.0 @PramodSSImmaneni please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex

[GitHub] apex-core pull request #415: APEXCORE-572 - Remove dependency on hadoop-comm...

2016-11-04 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/415 APEXCORE-572 - Remove dependency on hadoop-common test.jar @PramodSSImmaneni, @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-site pull request #51: Mentioned enforcing no trailing whitespaces by c...

2016-09-26 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-site/pull/51 Mentioned enforcing no trailing whitespaces by checkstyle. @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex

[GitHub] apex-core pull request #396: APEXCORE-538 - Log raw data when RPC message fa...

2016-09-22 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/396 APEXCORE-538 - Log raw data when RPC message fails to de-serialize @tweise Please review and cherry-pick. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #392: APEXCORE-535 - Node.teardown() should try to gr...

2016-09-14 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/392 APEXCORE-535 - Node.teardown() should try to gracefully shutdown exectutor service @tweise, @PramodSSImmaneni Please review. You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #389: APEXCORE-531 - Enable System.out/System.err che...

2016-09-13 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/389 APEXCORE-531 - Enable System.out/System.err check for *Test You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov/apex-core APEXCORE-531

[GitHub] apex-core pull request #388: APEXCORE-527 - Minor changes in LocalStramChild...

2016-09-13 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/388 APEXCORE-527 - Minor changes in LocalStramChildLauncher to help with unit test failures @tweise Please review You can merge this pull request into a Git repository by running: $ git pull

[GitHub] apex-core pull request #385: APEXCORE-522 - Promote singleton usage pattern ...

2016-09-08 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/385 --- 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] apex-core pull request #385: APEXCORE-522 - Promote singleton usage pattern ...

2016-09-08 Thread vrozov
GitHub user vrozov reopened a pull request: https://github.com/apache/apex-core/pull/385 APEXCORE-522 - Promote singleton usage pattern for String2String, Long2String and other StringCodecs You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #386: APEXCORE-524 - Add support for custom maven rep...

2016-09-08 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/386 APEXCORE-524 - Add support for custom maven repository to ClassPathResolverTest.testManifestClassPathResolver @tweise Please review You can merge this pull request into a Git repository by

[GitHub] apex-core pull request #385: APEXCORE-522 - Promote singleton usage pattern ...

2016-09-07 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/385 APEXCORE-522 - Promote singleton usage pattern for String2String, Long2String and other StringCodecs You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #384: APEXCORE-521 upate parent pom references.

2016-09-07 Thread vrozov
Github user vrozov closed the pull request at: https://github.com/apache/apex-core/pull/384 --- 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] apex-core pull request #384: APEXCORE-521 upate parent pom references.

2016-09-07 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/384 APEXCORE-521 upate parent pom references. test Apache Jenkins PR build. Please don't merge. You can merge this pull request into a Git repository by running: $ git pull https://githu

[GitHub] apex-core pull request #379: APEXCORE-518 - Update codestyle-config project ...

2016-09-02 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/379 APEXCORE-518 - Update codestyle-config project pom.xml @tweise Please pull in after #367 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] apex-core pull request #376: APEXCORE-516 - StramLocalCluster should always ...

2016-08-29 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/376 APEXCORE-516 - StramLocalCluster should always use loopback address for buffer server location @tweise Please review You can merge this pull request into a Git repository by running: $ git

[GitHub] apex-core pull request #375: APEXCORE-169 - instantiating DTLoggerFactory du...

2016-08-25 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/375 APEXCORE-169 - instantiating DTLoggerFactory during test causes incorrect logging behavior @tweise, @chandnisingh Please review You can merge this pull request into a Git repository by running

[GitHub] apex-core pull request #370: APEXCORE-502 Unnecessary byte array copy in Def...

2016-08-16 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/370 APEXCORE-502 Unnecessary byte array copy in DefaultKryoStreamCodec.toByteArray @tweise Please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #367: APEXCORE-506 Added check for trailing whitespac...

2016-08-12 Thread vrozov
GitHub user vrozov opened a pull request: https://github.com/apache/apex-core/pull/367 APEXCORE-506 Added check for trailing whitespaces @tweise, @chandnisingh Please review You can merge this pull request into a Git repository by running: $ git pull https://github.com/vrozov

[GitHub] apex-core issue #364: APEXCORE-448 Made operator name available in operator ...

2016-08-04 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/364 As far as I can see not all changes are correct (please see my comments regarding UnifierDeployInfo and access from two different threads to the operatorName). It will be good to have them fixed

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73454311 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/StreamingContainer.java --- @@ -902,14 +902,14 @@ private void deployNodes(List nodeList

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73454152 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73453695 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,112 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73451447 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,112 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73450932 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73450445 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/StreamingContainer.java --- @@ -902,14 +902,14 @@ private void deployNodes(List nodeList

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73449937 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/StreamingContainer.java --- @@ -902,14 +902,14 @@ private void deployNodes(List nodeList

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73448436 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73445463 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73437878 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73431751 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core issue #343: Update code style structure and docs

2016-08-03 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/343 @chandnisingh changes for IntelliJ looks good to me. Can you please review other IDE and merge if they look good to you. --- If your project is set up for it, you can reply to this email and have

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73426924 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73425701 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73423271 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-03 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73365188 --- Diff: engine/src/test/java/com/datatorrent/stram/engine/OperatorContextTest.java --- @@ -0,0 +1,99 @@ +/** + * Licensed to the Apache Software

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-01 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73068052 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-01 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73066955 --- Diff: engine/src/main/java/com/datatorrent/stram/engine/OperatorContext.java --- @@ -84,12 +87,13 @@ public void setIdleTimeout(long idleTimeout

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-01 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73066772 --- Diff: api/src/main/java/com/datatorrent/api/Context.java --- @@ -328,6 +328,8 @@ */ int getWindowsFromCheckpoint

[GitHub] apex-core pull request #364: APEXCORE-448 Made operator name available in op...

2016-08-01 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/364#discussion_r73065873 --- Diff: api/src/main/java/com/datatorrent/api/Context.java --- @@ -328,6 +328,8 @@ */ int getWindowsFromCheckpoint

[GitHub] apex-core pull request #358: *Review only* [APEXCORE-482]Make the downstream...

2016-07-28 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/358#discussion_r72615867 --- Diff: engine/src/main/java/com/datatorrent/stram/plan/physical/PTOperator.java --- @@ -85,6 +85,7 @@ private static final long

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

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

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

2016-07-22 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/359 @chaithu14 Please close this PR and open a new one once the issue is fixed. --- 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] apex-core pull request #361: APEXCORE-496 make operator name available to St...

2016-07-22 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/apex-core/pull/361#discussion_r71943358 --- Diff: api/src/main/java/com/datatorrent/api/StatsListener.java --- @@ -113,6 +113,8 @@ long getLatencyMA(); List

[GitHub] apex-core issue #358: *Review only* [APEXCORE-482]Make the downstream operat...

2016-07-22 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/358 Most of the time default unifier (the one that passes tuples unmodified from input streams to an output stream) is used. For that case, it is necessary to benchmark the change (please see my

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

2016-07-22 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/359 @chaithu14 As @amberarrow mentioned, definePartitions() may return superset or subset of current partitions. The bug seems to be in the redeploying dependencies, not in the modified code that

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

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

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

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

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

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

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

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

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

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

[GitHub] apex-core issue #358: *Review only* [APEXCORE-482]Make the downstream operat...

2016-07-15 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/358 What is the reason unifier does not have it's own StreamMeta? IMO, it will be better to introduce separate StreamMeta for unifiers instead of the locality field that is used only for uni

[GitHub] apex-core issue #358: *Review only* [APEXCORE-482]Make the downstream operat...

2016-07-15 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/apex-core/pull/358 TODO: provide benchmark numbers for 4, 16, 32 and possibly 64 unifiers comparing existing behavior with behavior after the commit. --- If your project is set up for it, you can reply to this

  1   2   >