[GitHub] apex-core pull request #452: APEXCORE-618 #resolve Added com.sun.jersey.clie...

2017-01-21 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-core/pull/452 --- 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-core pull request #452: APEXCORE-618 #resolve Added com.sun.jersey.clie...

2017-01-21 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-core/pull/452 APEXCORE-618 #resolve Added com.sun.jersey.client.apache4.ApacheHttpC… …lient4Handler.class in the bundled jars on app launch You can merge this pull request into a Git repository by

[GitHub] apex-core pull request #427: APEXCORE-585 #resolve do not try to calculate l...

2016-12-02 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-core/pull/427 APEXCORE-585 #resolve do not try to calculate latency if the first wi… …ndow has not complete @vrozov please review and merge You can merge this pull request into a Git

[GitHub] apex-malhar pull request #519: APEXMALHAR-2362 #resolve clearing the removed...

2016-11-30 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/519 APEXMALHAR-2362 #resolve clearing the removedSets at endWindow in SpillableSetMultimapImpl @brightchen please review and merge You can merge this pull request into a Git repository by

[GitHub] apex-malhar pull request #517: APEXMALHAR-2343 Count fix and accumulation in...

2016-11-29 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/517 APEXMALHAR-2343 Count fix and accumulation input parameter type change @brightchen @chinmaykolhatkar @siyuanh Please have a look You can merge this pull request into a Git repository by

[GitHub] apex-malhar pull request #516: REVIEW ONLY: Purging time buckets in Windowed...

2016-11-28 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/516 REVIEW ONLY: Purging time buckets in WindowedStorage @siyuanh Please review. This is based on the other PR you have #503. I'll add unit test for purging if this looks good. You can

[GitHub] apex-core pull request #336: APEXCORE-462 Cache the invalid web services inf...

2016-11-23 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-core/pull/336 --- 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 #465: REVIEW ONLY: changes to serde for managed sta...

2016-10-22 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/465 REVIEW ONLY: changes to serde for managed state and spillable ds @brightchen @tweise @siyuanh please review You can merge this pull request into a Git repository by running: $ git

[GitHub] apex-malhar pull request #462: APEXMALHAR-2305 changed javadoc of SessionWin...

2016-10-20 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/462 APEXMALHAR-2305 changed javadoc of SessionWindows to reflect the impl… …ementation change of SessionWindows @siyuanh please merge You can merge this pull request into a Git

[GitHub] apex-malhar pull request #460: APEXMALHAR-2305 updated documentation on Sess...

2016-10-20 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/460 APEXMALHAR-2305 updated documentation on SessionWindow to reflect the… … new proto-session window semantics @siyuanh please merge You can merge this pull request into a Git

[GitHub] apex-malhar pull request #458: APEXMALHAR-2305 #resolve Mirror the proto-ses...

2016-10-18 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/458 APEXMALHAR-2305 #resolve Mirror the proto-session window behavior des… …cribed in the streaming 102 blog @siyuanh please review and merge You can merge this pull request into

[GitHub] apex-malhar pull request #450: APEXMALHAR-2282 First iteration for windowed ...

2016-10-11 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/450 APEXMALHAR-2282 First iteration for windowed operator documentation @tweise please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-malhar pull request #449: APEXMALHAR-2280 First iteration for windowed ...

2016-10-11 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/449 --- 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

[GitHub] apex-malhar pull request #449: APEXMALHAR-2280 First iteration for windowed ...

2016-10-11 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/449 APEXMALHAR-2280 First iteration for windowed operator documentation @tweise please review You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-core pull request #409: APEXCORE-552 #resolve added support for applica...

2016-10-11 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-core/pull/409 APEXCORE-552 #resolve added support for application tags @PramodSSImmaneni please merge You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] apex-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-05 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-core/pull/404 --- 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-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-05 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-core/pull/404 APEXCORE-536 #resolve Upgrade Hadoop dependency version from 2.2.0 to 2.6.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] apex-malhar pull request #442: APEXMALHAR-2280 #resolve added package-info.j...

2016-10-05 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/442 APEXMALHAR-2280 #resolve added package-info.java to denote interface … …stability for all classes that belong to event window processing @siyuanh please merge You can merge

[GitHub] apex-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-04 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-core/pull/404 APEXCORE-536 #resolve Upgrade Hadoop dependency version from 2.2.0 to 2.6.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] apex-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-04 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-core/pull/404 --- 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-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-04 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-core/pull/404 APEXCORE-536 #resolve Upgrade Hadoop dependency version from 2.2.0 to 2.6.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] apex-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-04 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-core/pull/404 --- 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-core pull request #404: APEXCORE-536 #resolve Upgrade Hadoop dependency...

2016-10-04 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-core/pull/404 APEXCORE-536 #resolve Upgrade Hadoop dependency version from 2.2.0 to 2.6.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/davidyan74

[GitHub] apex-core pull request #401: APEXCORE-544 #resolve fixed latency calculation...

2016-09-30 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-core/pull/401 APEXCORE-544 #resolve fixed latency calculation when the upstream heartbeat is processed later than the downstream for a particular window @tweise please review. this fixes the problem with

[GitHub] apex-malhar pull request #436: APEXMALHAR-2273 #resolve do not fire retracti...

2016-09-30 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/436 APEXMALHAR-2273 #resolve do not fire retraction trigger if retraction… … value is same as current value and fireOnlyUpdatedPanes is true @siyuanh please merge You can merge

[GitHub] apex-malhar pull request #431: APEXMALHAR-2267 #resolve renamed spillable da...

2016-09-27 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-malhar/pull/431 APEXMALHAR-2267 #resolve renamed spillable data structures to remove the word "Byte" @siyuanh please merge You can merge this pull request into a Git repository by running:

[GitHub] apex-malhar pull request #431: APEXMALHAR-2267 #resolve renamed spillable da...

2016-09-27 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/431 --- 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

[GitHub] apex-malhar pull request #431: APEXMALHAR-2267 #resolve renamed spillable da...

2016-09-27 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/431 APEXMALHAR-2267 #resolve renamed spillable data structures to remove the word "Byte" @siyuanh please merge You can merge this pull request into a Git repository by running:

[GitHub] apex-malhar pull request #424: APEXMALHAR-2130 Spillable implementation for ...

2016-09-27 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-malhar/pull/424 APEXMALHAR-2130 Spillable implementation for WindowedOperator @tweise @brightchen @siyuanh Please review and merge You can merge this pull request into a Git repository by running

[GitHub] apex-malhar pull request #424: APEXMALHAR-2130 Spillable implementation for ...

2016-09-27 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/424 --- 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

[GitHub] apex-malhar pull request #424: APEXMALHAR-2130 Spillable implementation for ...

2016-09-22 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/424 APEXMALHAR-2130 Spillable implementation for WindowedOperator @tweise @brightchen @siyuanh Please review and merge You can merge this pull request into a Git repository by running

[GitHub] apex-malhar pull request #345: APEXMALHAR-2130 REVIEW ONLY (WindowedOperator...

2016-09-22 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/345 --- 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

[GitHub] apex-malhar pull request #377: *Review Only* APEXMALHAR-2192 Added some spil...

2016-09-22 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/377 --- 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

[GitHub] apex-malhar pull request #399: REVIEW ONLY: exposing seeking and iterating o...

2016-09-22 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/399 --- 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

[GitHub] apex-malhar pull request #405: APEXMALHAR-2130 #resolve Added a spillable ma...

2016-09-20 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/405 --- 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

[GitHub] apex-malhar pull request #418: APEXMALHAR-2247 #resolve Added iteration feat...

2016-09-19 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/418 APEXMALHAR-2247 #resolve Added iteration feature in SpillableArrayListImpl and generalize SerdeListSlice to SerdeCollectionSlice @siyuanh please review and merge You can merge this pull

[GitHub] apex-malhar pull request #417: APEXMALHAR-2246 #resolve use Slice instead of...

2016-09-19 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/417 APEXMALHAR-2246 #resolve use Slice instead of byte[] in the underlying map of SpillableByteArrayListMultimapImpl @siyuanh Please review and merge You can merge this pull request into a Git

[GitHub] apex-malhar pull request #416: APEXMALHAR-2245 #resolve Add the key in remov...

2016-09-19 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/416 APEXMALHAR-2245 #resolve Add the key in removedKeys even if the key does not appear in the cache @siyuanh Please review and merge You can merge this pull request into a Git repository by

[GitHub] apex-malhar pull request #408: APEXMALHAR-2130 Added SpillableSet and Spilla...

2016-09-12 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/408 APEXMALHAR-2130 Added SpillableSet and SpillableSetMultimap implementations @tweise @siyuanh @ilooner please review You can merge this pull request into a Git repository by running

[GitHub] apex-malhar pull request #407: APEXMALHAR-2130 WindowStorage interface chang...

2016-09-11 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/407 APEXMALHAR-2130 WindowStorage interface changes in preparation of inc… …orporating spillable data structures @siyuanh please review and merge @tweise @ShunxinLu FYI You can

[GitHub] apex-malhar pull request #405: APEXMALHAR-2130 #resolve Added a spillable ma...

2016-09-08 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/405 APEXMALHAR-2130 #resolve Added a spillable map that takes two keys with support of iterating through all entries with a given first key @siyuanh @tweise @ilooner Please review. This is

[GitHub] apex-malhar pull request #403: APEXMALHAR-2229 #resolve Added peek and hasNe...

2016-09-07 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/403 APEXMALHAR-2229 #resolve Added peek and hasNext methods to FileAccess interface @tweise please review You can merge this pull request into a Git repository by running: $ git pull

[GitHub] apex-malhar pull request #399: REVIEW ONLY: exposing seeking and iterating o...

2016-09-02 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/399 REVIEW ONLY: exposing seeking and iterating of TFile in ManagedState @chandnisingh @ilooner @tweise @siyuanh Feedback is appreciated. I'll add the missing unit tests later so hold

[GitHub] apex-malhar pull request #391: APEXMALHAR-2215 #resolve replace Gutenberg co...

2016-08-30 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/391 APEXMALHAR-2215 #resolve replace Gutenberg content with Apache web si… …te content because of license concern @tweise please review You can merge this pull request into a Git

[GitHub] apex-malhar pull request #377: *Review Only* APEXMALHAR-2192 Added some spil...

2016-08-23 Thread davidyan74
GitHub user davidyan74 opened a pull request: https://github.com/apache/apex-malhar/pull/377 *Review Only* APEXMALHAR-2192 Added some spillable method implementations This is not ready for merge yet. Please do not comment on missing unit tests, code style, debug statements, etc

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-08 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73941677 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/WindowListener.java --- @@ -0,0 +1,39 @@ +/** + * Licensed to the

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73609127 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -180,4 +188,40 @@ public void

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73589696 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -180,4 +188,40 @@ public void

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73583092 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -53,16 +56,16 @@ private static File configFile

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73582886 --- Diff: engine/src/test/resources/testConfigPackage/testConfigPackageSrc/app/testApp.json --- @@ -0,0 +1 @@ +{"displayName"

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73582758 --- Diff: engine/src/test/resources/testAppPackage/mydtapp/src/main/resources/com/example/mydtapp/Application/ExtRes.html --- @@ -0,0 +1,21

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73582496 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -126,10 +130,14 @@ public void testLaunchAppPackagePropertyPrecedence

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73582122 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -103,8 +107,8 @@ public static void finished() public void

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73581887 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -180,4 +188,40 @@ public void

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73581539 --- Diff: engine/src/test/java/com/datatorrent/stram/cli/ApexCliTest.java --- @@ -180,4 +188,40 @@ public void

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73581176 --- Diff: engine/src/main/java/com/datatorrent/stram/client/AppPackage.java --- @@ -49,6 +49,8 @@ import net.lingala.zip4j.exception.ZipException

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-08-04 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r73580877 --- Diff: apex-conf-archetype/src/main/resources/archetype-resources/pom.xml --- @@ -25,6 +25,7 @@ 1.. classpath

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73079896 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73079738 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73075544 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73075245 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73073791 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteArrayListMultimapImpl.java --- @@ -0,0 +1,292

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73073399 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031805 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableComplexComponentImpl.java --- @@ -0,0 +1,202

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032806 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/utils/serde/SerdeListSliceTest.java --- @@ -16,29 +16,33 @@ * specific language

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032779 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/inmem/InMemorySpillableStateStoreTest.java --- @@ -0,0 +1,63

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032752 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/WindowBoundedMapCacheTest.java --- @@ -0,0 +1,119

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032644 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SpillableComplexComponentImplTest.java --- @@ -0,0 +1,88

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032673 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SpillableTestUtils.java --- @@ -0,0 +1,134 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032713 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/TimeBasedPriorityQueueTest.java --- @@ -0,0 +1,134

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032590 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteMapImplTest.java --- @@ -0,0 +1,487

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032473 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SpillableArrayListImplTest.java --- @@ -0,0 +1,597

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032527 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteArrayListMultimapImplTest.java --- @@ -0,0 +1,344

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032414 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/state/spillable/SequentialSpillableIdentifierGeneratorTest.java --- @@ -0,0 +1,128

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032354 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/SliceUtils.java --- @@ -0,0 +1,70 @@ +/** + * Licensed to the Apache

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032285 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/SerdeStringSlice.java --- @@ -0,0 +1,51 @@ +/** + * Licensed to the

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032245 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/SerdeListSlice.java --- @@ -0,0 +1,103 @@ +/** + * Licensed to the

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032214 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/SerdeIntSlice.java --- @@ -0,0 +1,50 @@ +/** + * Licensed to the

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032134 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/PassThruByteArraySliceSerde.java --- @@ -0,0 +1,55

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032033 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/inmem/InMemSpillableStateStore.java --- @@ -0,0 +1,116

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032183 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/utils/serde/PassThruSliceSerde.java --- @@ -16,30 +16,32 @@ * specific language

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73032099 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/managed/ManagedStateSpillableStateStore.java --- @@ -16,29 +16,19

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031950 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/WindowBoundedMapCache.java --- @@ -0,0 +1,119

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031909 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/TimeBasedPriorityQueue.java --- @@ -0,0 +1,128 @@ +package

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031870 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableStateStore.java --- @@ -0,0 +1,33 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031644 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SequentialSpillableIdentifierGenerator.java --- @@ -0,0 +1,78

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031608 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteMapImpl.java --- @@ -0,0 +1,237 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-08-01 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r73031476 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteArrayListMultimapImpl.java --- @@ -0,0 +1,292

[GitHub] apex-malhar pull request #345: APEXMALHAR-2130 REVIEW ONLY (WindowedOperator...

2016-08-01 Thread davidyan74
GitHub user davidyan74 reopened a pull request: https://github.com/apache/apex-malhar/pull/345 APEXMALHAR-2130 REVIEW ONLY (WindowedOperator): incorporating Spillable data structures You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] apex-malhar pull request #345: APEXMALHAR-2130 REVIEW ONLY (WindowedOperator...

2016-08-01 Thread davidyan74
Github user davidyan74 closed the pull request at: https://github.com/apache/apex-malhar/pull/345 --- 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

[GitHub] apex-core issue #363: Documentation for secure impersonation

2016-07-28 Thread davidyan74
Github user davidyan74 commented on the issue: https://github.com/apache/apex-core/pull/363 Looks good to me. Will merge once it's squashed --- 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] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-07-28 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r72670575 --- Diff: engine/src/test/resources/testOperators/src/main/java/org/apache/apex/test/DevNull.java --- @@ -16,25 +16,19 @@ * specific language

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-07-28 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r72670501 --- Diff: engine/src/test/resources/testOperators/src/main/java/org/apache/apex/test/RandomGen.java --- @@ -16,24 +16,44 @@ * specific language

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-07-28 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r72670240 --- Diff: engine/src/test/java/com/datatorrent/stram/support/StramTestSupport.java --- @@ -157,21 +160,42 @@ public static void waitForWindowComplete

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-07-28 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r72670019 --- Diff: engine/src/main/java/com/datatorrent/stram/client/ConfigPackage.java --- @@ -71,6 +72,7 @@ private final Map properties = new

[GitHub] apex-core pull request #360: APEXCORE-495 supporting apps in config package.

2016-07-28 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-core/pull/360#discussion_r72669919 --- Diff: engine/src/main/java/com/datatorrent/stram/cli/ApexCli.java --- @@ -3850,6 +3908,9 @@ private Option add(Option opt) final Option

[GitHub] apex-malhar pull request #345: APEXMALHAR-2130 REVIEW ONLY (WindowedOperator...

2016-07-27 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/345#discussion_r72521168 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/window/SpillableWindowedStorageTest.java --- @@ -0,0 +1,122 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #345: APEXMALHAR-2130 REVIEW ONLY (WindowedOperator...

2016-07-27 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/345#discussion_r72494611 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/window/impl/SpillableWindowedPlainStorage.java --- @@ -0,0 +1,188

[GitHub] apex-malhar pull request #345: REVIEW ONLY (WindowedOperator): incorporating...

2016-07-26 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/345#discussion_r72309272 --- Diff: library/src/test/java/org/apache/apex/malhar/lib/window/SpillableWindowedStorageTest.java --- @@ -0,0 +1,122 @@ +/** + * Licensed

[GitHub] apex-malhar pull request #324: Spillable Datastructures PR for review only

2016-07-26 Thread davidyan74
Github user davidyan74 commented on a diff in the pull request: https://github.com/apache/apex-malhar/pull/324#discussion_r72308835 --- Diff: library/src/main/java/org/apache/apex/malhar/lib/state/spillable/SpillableByteArrayListMultimapImpl.java --- @@ -0,0 +1,262

  1   2   >