[GitHub] incubator-pirk pull request #82: [PIRK-58] - Improve Paillier Test Speed

2016-08-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/82 --- 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] incubator-pirk issue #82: [PIRK-58] - Improve Paillier Test Speed

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/82 lol - will merge now :) --- 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

[GitHub] incubator-pirk issue #82: [PIRK-58] - Improve Paillier Test Speed

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/82 This is a non-material change that yields increased test speed for Paillier - in practice, we would never use a modulus up to 7000 bits --- If your project is set up for it, you can

[GitHub] incubator-pirk pull request #82: [PIRK-58] - Improve Paillier Test Speed

2016-08-27 Thread ellisonanne
GitHub user ellisonanne opened a pull request: https://github.com/apache/incubator-pirk/pull/82 [PIRK-58] - Improve Paillier Test Speed (note that I accidentally named the branch pirk-59 instead of pirk-58) You can merge this pull request into a Git repository by running: $

Re: [VOTE] Apache Pirk 0.1.0-incubating Release

2016-08-27 Thread Ellison Anne Williams
Looks like you responded to dev@ - did you mean to respond to general@incubator? On Sat, Aug 27, 2016 at 11:13 AM, Tim Ellison wrote: > +1 binding. > > Regards, > Tim > > > On 26/08/16 13:37, Ellison Anne Williams wrote: > > Hi All, > > > > The PPMC vote for the Apache

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/81 --- 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] incubator-pirk pull request #76: [PIRK-21] - Initial Spark Streaming Respond...

2016-08-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/76 --- 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

Re: [VOTE] Apache Pirk 0.1.0-incubating Release

2016-08-27 Thread Tim Ellison
+1 binding. Regards, Tim On 26/08/16 13:37, Ellison Anne Williams wrote: > Hi All, > > The PPMC vote for the Apache Pirk 0.1.0-incubating release has passed after > updating the cryptographic export information; it is a source-only release > (no binary artifacts). We kindly request that the

[GitHub] incubator-pirk issue #76: [PIRK-21] - Initial Spark Streaming Responder Impl...

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/76 Merging now - will keep working on it via JIRA issues and PRs --- 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-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread tellison
Github user tellison commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76517695 --- Diff: src/main/java/org/apache/pirk/querier/wideskies/encrypt/EncryptQuery.java --- @@ -163,8 +144,8 @@ public void encrypt(int numThreads)

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread tellison
Github user tellison commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76517506 --- Diff: src/main/java/org/apache/pirk/querier/wideskies/encrypt/EncryptQuery.java --- @@ -56,103 +56,84 @@ { private static final

[GitHub] incubator-pirk issue #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/81 +1 very nice - I made a few nitty comments about javadocs... --- 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-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76517354 --- Diff: src/main/java/org/apache/pirk/querier/wideskies/encrypt/EncryptQuery.java --- @@ -200,31 +181,29 @@ public void encrypt(int numThreads)

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76517343 --- Diff: src/main/java/org/apache/pirk/querier/wideskies/encrypt/EncryptQuery.java --- @@ -163,8 +144,8 @@ public void encrypt(int numThreads)

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread ellisonanne
Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76517300 --- Diff: src/main/java/org/apache/pirk/querier/wideskies/encrypt/EncryptQuery.java --- @@ -56,103 +56,84 @@ { private static final

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread tellison
Github user tellison commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76516765 --- Diff: src/main/java/org/apache/pirk/schema/data/partitioner/PrimitiveTypePartitioner.java --- @@ -32,7 +32,7 @@ * Class for partitioning

[GitHub] incubator-pirk pull request #74: [PIRK-4] Add Streaming Implementation for A...

2016-08-27 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/74#discussion_r76515132 --- Diff: src/main/java/org/apache/pirk/responder/wideskies/storm/EncRowCalcBolt.java --- @@ -0,0 +1,243 @@ +/* + * Licensed to the Apache

[GitHub] incubator-pirk pull request #74: [PIRK-4] Add Streaming Implementation for A...

2016-08-27 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/74#discussion_r76515127 --- Diff: src/main/java/org/apache/pirk/responder/wideskies/storm/EncColMultBolt.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the Apache

[GitHub] incubator-pirk pull request #74: [PIRK-4] Add Streaming Implementation for A...

2016-08-27 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/74#discussion_r76515120 --- Diff: src/main/java/org/apache/pirk/responder/wideskies/common/ComputeEncryptedRow.java --- @@ -321,4 +316,81 @@ public static void

[GitHub] incubator-pirk pull request #74: [PIRK-4] Add Streaming Implementation for A...

2016-08-27 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/74#discussion_r76515109 --- Diff: src/main/java/org/apache/pirk/responder/wideskies/common/ComputeEncryptedRow.java --- @@ -321,4 +316,81 @@ public static void

[GitHub] incubator-pirk pull request #81: Enhancements to Wideskies encrypt/decrypt.

2016-08-27 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/81#discussion_r76514978 --- Diff: src/main/java/org/apache/pirk/schema/data/partitioner/PrimitiveTypePartitioner.java --- @@ -32,7 +32,7 @@ * Class for partitioning