Ideas to speed up PaillierTest ?

2016-07-31 Thread Suneel Marthi
Any ideas on how to speed up PaillierTest? Is the bottleneck in BigInter.multiply(BigInteger) or is the KeyGeneration ?

[GitHub] incubator-pirk pull request #40: [nitpick]: change the forkCount to reliably...

2016-07-31 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/incubator-pirk/pull/40 [nitpick]: change the forkCount to reliably execute tests in parallel… … and set reuseForks to False. You can merge this pull request into a Git repository by running: $ git pull htt

[GitHub] incubator-pirk pull request #39: [PIRK-37]: Rename package for test classes ...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/39 --- 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 #39: [PIRK-37]: Rename package for test classes to be o...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/39 +1 Will merge now and update 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 project does not have this fea

[GitHub] incubator-pirk issue #39: [PIRK-37]: Rename package for test classes to be o...

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/39 this is good to merge. have not updated the generated test docs as part of this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] incubator-pirk pull request #39: [PIRK-37]: Rename package for test classes ...

2016-07-31 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/incubator-pirk/pull/39 [PIRK-37]: Rename package for test classes to be org.apache.pirk.* You can merge this pull request into a Git repository by running: $ git pull https://github.com/smarthi/incubator-pirk

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/38 --- 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 #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/38 Good catch Suneel! --- 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 wi

Re: Javadocs for Query/Data schema loaders update?

2016-07-31 Thread Ellison Anne Williams
Go for it Walter! On Sun, Jul 31, 2016 at 12:38 PM, Walter Ray-Dulany wrote: > I've been reading the DataSchemaLoader > (src/main/java/org/apache/pirk/schema/data/DataSchemaLoader.java). I > noticed that the loader will only mark an element as an array if the value > of the tag in the data sche

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/38 We have to change the package names for test classes from present 'test.xx' to 'org.apache.pirk.xx'. That would be a different jira. --- If your project is set up for it, you can reply to th

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/38 Lgtm +1 Ship it!! --- 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

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72909521 --- Diff: src/test/java/test/general/PartitionUtilsTest.java --- @@ -39,7 +40,8 @@ { private static final Logger logger = LoggerFactory

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72909520 --- Diff: src/test/java/test/wideskies/standalone/StandaloneTest.java --- @@ -45,10 +49,17 @@ private static final String STOPLIST_FILE

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72909436 --- Diff: src/test/java/test/general/PartitionUtilsTest.java --- @@ -39,7 +40,8 @@ { private static final Logger logger = LoggerFactory.get

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72909418 --- Diff: src/test/java/test/wideskies/standalone/StandaloneTest.java --- @@ -45,10 +49,17 @@ private static final String STOPLIST_FILE = "

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/38 Done - pls review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wis

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/38 I have a fix at https://github.com/smarthi/incubator-pirk/commit/e2c76301b2345ca7371af8b10fedb697182dbf2c --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/38 Yes - this can be done throughout --- 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

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/38 What's presently in the Constructor() can be split between methods annotated with @BeforeClass and @AfterClass. There's no need of a constructor then. --- If your project is set up for it,

[GitHub] incubator-pirk issue #38: [PIRK-36]: Ensure Schema Properties and Registries...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/38 If we are going to move it to a method, I would rather have it in a setup() method so that it is self-contained - i.e., that a test does not rely on previous tests cleaning up after themse

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72907964 --- Diff: src/test/java/test/wideskies/standalone/StandaloneTest.java --- @@ -49,6 +51,12 @@ public StandaloneTest() throws Exception

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread smarthi
Github user smarthi commented on a diff in the pull request: https://github.com/apache/incubator-pirk/pull/38#discussion_r72907957 --- Diff: src/test/java/test/wideskies/standalone/StandaloneTest.java --- @@ -49,6 +51,12 @@ public StandaloneTest() throws Exception

[GitHub] incubator-pirk pull request #38: [PIRK-36]: Ensure Schema Properties and Reg...

2016-07-31 Thread ellisonanne
GitHub user ellisonanne opened a pull request: https://github.com/apache/incubator-pirk/pull/38 [PIRK-36]: Ensure Schema Properties and Registries are Reset at the Beginning of Relevant Tests You can merge this pull request into a Git repository by running: $ git pull https:

[GitHub] incubator-pirk pull request #37: [PIRK-35]: Execute tests in Parallel

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/37 --- 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 #37: [PIRK-35]: Execute tests in Parallel

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/37 Will do --- 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

[GitHub] incubator-pirk issue #37: [PIRK-35]: Execute tests in Parallel

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/37 Sure. could u be create a different Jira for that? --- 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 hav

[GitHub] incubator-pirk issue #37: [PIRK-35]: Execute tests in Parallel

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/37 +1 For cleanliness, we should also make sure that the schema properties are reset and the schema registries are reset at the beginning of all tests that access/utilize schemas. I

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Ellison Anne Williams
Sure - for cleanliness, we should also make sure that the schema properties are reset and the schema registries are reset at the beginning of all tests that access/utilize schemas. I only put it on top of the QueryParserUtilsTest, but it also should be done to the StandaloneTest, LoadQuerySchemaTes

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Suneel Marthi
Can the tests then be configured to run in parallel independently given this ? On Sun, Jul 31, 2016 at 1:04 PM, Ellison Anne Williams < eawilliamsp...@gmail.com> wrote: > So, I reset the query.schemas and data.schemas properties before creating > the new schemas in the QueryParserUtilsTest and th

[GitHub] incubator-pirk pull request #37: [PIRK-35]: Execute tests in Parallel

2016-07-31 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/incubator-pirk/pull/37 [PIRK-35]: Execute tests in Parallel You can merge this pull request into a Git repository by running: $ git pull https://github.com/smarthi/incubator-pirk PIRK-35 Alternatively you ca

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Ellison Anne Williams
So, I reset the query.schemas and data.schemas properties before creating the new schemas in the QueryParserUtilsTest and the jenkins build is now fixed. Turns out it was a good, old fashioned execution ordering bug. If the StandaloneTest is executed first, it populates the query.schemas with Quer

Jenkins build is back to stable : pirk » Apache Pirk (incubating) Project #22

2016-07-31 Thread Apache Jenkins Server
See

Jenkins build is back to stable : pirk #22

2016-07-31 Thread Apache Jenkins Server
See

[GitHub] incubator-pirk pull request #36: Trivial fix to remove github footer from po...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/36 --- 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 #36: Trivial fix to remove github footer from pom.xml

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/36 Sounds good --- 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

[GitHub] incubator-pirk issue #36: Trivial fix to remove github footer from pom.xml

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/36 I am gonna go ahead and merge this, this is very trivial and bad fingering on my part. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

Javadocs for Query/Data schema loaders update?

2016-07-31 Thread Walter Ray-Dulany
I've been reading the DataSchemaLoader (src/main/java/org/apache/pirk/schema/data/DataSchemaLoader.java). I noticed that the loader will only mark an element as an array if the value of the tag in the data schema xml file is either (a) empty or (b) equal to "true" after trimming and lowercasing.

Jenkins build is still unstable: pirk » Apache Pirk (incubating) Project #21

2016-07-31 Thread Apache Jenkins Server
See

Jenkins build is still unstable: pirk #21

2016-07-31 Thread Apache Jenkins Server
See

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Ellison Anne Williams
No - it looks like it may have something to do with the query/data schema registries not being cleared before starting the QueryParserUtilsTest (i.e. leftover references to query schemas using a StopListFilter) - I am putting in a fix now, will push it up, and see if it is OK by Jenkins (if not, I

Jenkins build is still unstable: pirk #20

2016-07-31 Thread Apache Jenkins Server
See

Jenkins build is still unstable: pirk » Apache Pirk (incubating) Project #20

2016-07-31 Thread Apache Jenkins Server
See

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Suneel Marthi
Should the stopListFile be in the jar ? On Sun, Jul 31, 2016 at 11:47 AM, Ellison Anne Williams < eawilliamsp...@gmail.com> wrote: > I updated the Jenkins build info from JDK 1.7 to 1.8. It now builds, but > has issues with finding the stopListFile in the QueryParserUtilsTest. I > will investigat

[GitHub] incubator-pirk issue #36: Trivial fix to remove github footer from pom.xml

2016-07-31 Thread smarthi
Github user smarthi commented on the issue: https://github.com/apache/incubator-pirk/pull/36 Not sure, how that made it thru. --- 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 ena

[GitHub] incubator-pirk pull request #36: Trivial fix to remove github footer from po...

2016-07-31 Thread smarthi
GitHub user smarthi opened a pull request: https://github.com/apache/incubator-pirk/pull/36 Trivial fix to remove github footer from pom.xml You can merge this pull request into a Git repository by running: $ git pull https://github.com/smarthi/incubator-pirk sundry Alternati

Jenkins build is still unstable: pirk #19

2016-07-31 Thread Apache Jenkins Server
See

Jenkins build is still unstable: pirk » Apache Pirk (incubating) Project #19

2016-07-31 Thread Apache Jenkins Server
See

[GitHub] incubator-pirk pull request #33: [PIRK-28]: Logger Statements Not Working in...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-pirk/pull/33 --- 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 #33: [PIRK-28]: Logger Statements Not Working in/from D...

2016-07-31 Thread ellisonanne
Github user ellisonanne commented on the issue: https://github.com/apache/incubator-pirk/pull/33 +1 Thanks for tracking this down! --- 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

Re: Build failed in Jenkins: pirk #17

2016-07-31 Thread Ellison Anne Williams
I updated the Jenkins build info from JDK 1.7 to 1.8. It now builds, but has issues with finding the stopListFile in the QueryParserUtilsTest. I will investigate... On Sat, Jul 30, 2016 at 10:50 PM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See

Jenkins build is unstable: pirk » Apache Pirk (incubating) Project #18

2016-07-31 Thread Apache Jenkins Server
See

Jenkins build is unstable: pirk #18

2016-07-31 Thread Apache Jenkins Server
See

Podling Report Reminder - August 2016

2016-07-31 Thread johndament
Dear podling, This email was sent by an automated system on behalf of the Apache Incubator PMC. It is an initial reminder to give you plenty of time to prepare your quarterly board report. The board meeting is scheduled for Wed, 17 August 2016, 10:30 am PDT. The report for your podling will form