[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 a

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

2016-08-27 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/82 +1 - anything that can speed up Paillier and prevent myself from reaching out to the next can is a Big 👍 --- If your project is set up for it, you can reply to this email and have your r

[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 reply

[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: $ gi

Re: [VOTE] Apache Pirk 0.1.0-incubating Release

2016-08-27 Thread Tim Ellison
On 27/08/16 16:26, Ellison Anne Williams wrote: > Looks like you responded to dev@ - did you mean to respond to > general@incubator? Groan, thanks Ellison Anne. In my defence, the vote's "Reply-to:" is set to this list ;-) -- but I should have checked it! Regards, Tim > On Sat, Aug 27, 2016 at

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 Pirk 0.1.0-incubating rele

[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 IPM

[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 great - thanks - merging 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 featur

[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 projec

[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) throws

[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 Logger

[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 projec

[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) t

[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) thr

[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 Log

[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 ob

[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 Soft

[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 Soft

[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 loadCacheFromH

[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 loadCacheFromH

[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_r76515097 --- Diff: src/main/java/org/apache/pirk/responder/wideskies/ResponderProps.java --- @@ -90,7 +127,7 @@ static boolean validateResponderProperties()

[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 obj