[GitHub] incubator-rya issue #299: RYA-500: Make RdfFileInputTool to accept multiple ...

2018-07-04 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/299 Asfbot build ---

[GitHub] incubator-rya pull request #248: RYA-356 Added a Twill App for running the p...

2017-11-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/248#discussion_r148799979 --- Diff: extras/periodic.notification/api/src/main/java/org/apache/rya/periodic/notification/api/PeriodicNotificationClient.java --- @@ -36,29

[GitHub] incubator-rya issue #236: RYA-393 MongoDB Batch Writer Duplicate Key Excepti...

2017-10-24 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/236 Asfbot build ---

[GitHub] incubator-rya issue #241: RYA-333 Added "-P coverage" to generate code cover...

2017-10-10 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/241 I tried creating a build on b.a.o with the coverage profile enabled and "record jacoco report" plug-in enabled. You can check on it's progress here: https://builds.apac

[GitHub] incubator-rya issue #236: RYA-393 MongoDB Batch Writer Duplicate Key Excepti...

2017-10-06 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/236 @ejwhite922 There's a couple issues with the batch writer and it doesn't have to do with exceptions being caught. (1) the QueueFullChecker thread should not be a daemon thread "The

[GitHub] incubator-rya issue #236: RYA-393 MongoDB Batch Writer Duplicate Key Excepti...

2017-10-05 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/236 The more I look at this, the more I’m confident that this has to more with the mongo libs and not anything that we wrote for Rya. I’ll try to put together an example tomorrow. ---

[GitHub] incubator-rya issue #236: RYA-393 MongoDB Batch Writer Duplicate Key Excepti...

2017-10-05 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/236 I'm going to hold off on merging this for a minute. This exception talk has me thinking there are other exceptions that we are not processing correctly. brb ---

[GitHub] incubator-rya pull request #219: Creating inference Examples

2017-09-07 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/219#discussion_r137566563 --- Diff: extras/indexingExample/src/main/java/InferenceExamples.java --- @@ -0,0 +1,592 @@ +import java.io.IOException; +import

[GitHub] incubator-rya pull request #219: Creating inference Examples

2017-09-07 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/219#discussion_r137567339 --- Diff: extras/indexingExample/src/main/java/InferenceExamples.java --- @@ -0,0 +1,592 @@ +import java.io.IOException; +import

[GitHub] incubator-rya pull request #219: Creating inference Examples

2017-09-07 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/219#discussion_r137566475 --- Diff: extras/indexingExample/src/main/java/InferenceExamples.java --- @@ -0,0 +1,592 @@ +import java.io.IOException; +import

[GitHub] incubator-rya pull request #213: RYA-246-Query-Export-Strategy

2017-08-23 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/213#discussion_r134795805 --- Diff: common/rya.api/src/main/java/org/apache/rya/api/client/CreatePCJ.java --- @@ -27,8 +29,28 @@ @DefaultAnnotation(NonNull.class

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/204#discussion_r133984224 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/iter/RyaStatementBindingSetCursorIterator.java --- @@ -44,20 +48,21

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/204#discussion_r133983577 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBQueryEngine.java --- @@ -140,72 +121,35 @@ public MongoDBRdfConfiguration

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/204#discussion_r133983396 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBQueryEngine.java --- @@ -140,72 +121,35 @@ public MongoDBRdfConfiguration

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/204#discussion_r133983248 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBQueryEngine.java --- @@ -111,25 +97,20 @@ public MongoDBRdfConfiguration

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/204#discussion_r133983122 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBQueryEngine.java --- @@ -86,22 +84,10 @@ public MongoDBRdfConfiguration

[GitHub] incubator-rya pull request #204: RYA-337 Adding batch queries to MongoDB. Cl...

2017-08-14 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/204 RYA-337 Adding batch queries to MongoDB. Closes #204 ## Description Added a batch query mechanism to MongoDB DAO and simplified MongoDBQueryEngine ### Tests Ran

[GitHub] incubator-rya issue #203: Fixes a regression with QueriesBenchmarkConfReader...

2017-08-11 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/203 Thanks. I'll merge it 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-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @isper3at 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 feature

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @pujav65 totally agree. That's why I added the "please get some metrics" portion. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132245115 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -64,14 +68,14 @@ @Override

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132242206 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -53,8 +54,11 @@ public static

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132235261 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -64,14 +68,14 @@ @Override

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132235567 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -64,14 +68,14 @@ @Override

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132235041 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -53,8 +54,11 @@ public static

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132191684 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -53,8 +54,11 @@ public static

[GitHub] incubator-rya pull request #199: RYA-316 Long OBJ string

2017-08-09 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/199#discussion_r132192953 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/dao/SimpleMongoDBStorageStrategy.java --- @@ -85,14 +89,14 @@ public DBObject

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @isper3at Since this was so straightforward, can you go ahead and hash the S and O as well? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-rya pull request #196: RYA-313

2017-08-04 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/196#discussion_r131464691 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBQueryEngine.java --- @@ -118,7 +124,8 @@ public MongoDBRdfConfiguration

[GitHub] incubator-rya issue #196: RYA-313

2017-08-04 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/196 Looks good @isper3at. I'm going to test this out and wait for one more before I merge. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-rya pull request #194: RYA-325 Renamed rya.console to rya.shell.

2017-08-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/194#discussion_r131287677 --- Diff: extras/pom.xml --- @@ -34,7 +34,7 @@ under the License. rya.prospector rya.manual

[GitHub] incubator-rya issue #181: Rya-307 MongoDB Rya DAO Batch Writer

2017-08-03 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/181 @isper3at Thanks for taking a look at it. I'm testing it out now before I merge it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-rya pull request #194: RYA-325 Renamed rya.console to rya.shell.

2017-08-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/194#discussion_r131240139 --- Diff: extras/pom.xml --- @@ -34,7 +34,7 @@ under the License. rya.prospector rya.manual

[GitHub] incubator-rya issue #181: Rya-307 MongoDB Rya DAO Batch Writer

2017-08-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/181 @pujav65 sounds good. We've been testing out #186 and that's about to be merged. Hold off on merging #181 until #186 (and perhaps #182) is on master. --- If your project is set up

[GitHub] incubator-rya issue #182: RYA-324, RYA-272 Geo refactoring and examples

2017-08-01 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/182 I know that we planned to commit this PR to master first, but #186 is ready to go. I looked at this and #186. It turns out that we can commit #186 into master, then commit

[GitHub] incubator-rya issue #190: RYA-330 Cleaned up build testing and added profile...

2017-08-01 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/190 Rebuilding on my local machine with this latest update. --- 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

[GitHub] incubator-rya issue #188: RYA-318 Rya Shell improvements for packaging, load...

2017-07-28 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/188 @jdasch `ERROR: Maven JVM terminated unexpectedly with exit code 137`. The build servers are dead to me. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-28 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 @pujav65 The link you sent pointed to a failed build (the grey bulb means it failed). Also, you're right, that configuration doesn't build optional. I kicked of a job earlier

[GitHub] incubator-rya issue #186: Rya 317 Added Near function for geo mongodb

2017-07-28 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/186 asfbot build --- 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-rya issue #186: Rya 317 Added Near function for geo mongodb

2017-07-28 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/186 asfbot build --- 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-rya pull request #188: RYA-318 Rya Shell improvements for packagin...

2017-07-28 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/188#discussion_r130093053 --- Diff: extras/indexing/src/main/java/org/apache/rya/api/client/accumulo/AccumuloExecuteSparqlQuery.java --- @@ -0,0 +1,150

[GitHub] incubator-rya issue #186: Rya 317 Added Near function for geo mongodb

2017-07-28 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/186 asfbot build --- 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-rya pull request #188: RYA-318 Rya Shell improvements for packagin...

2017-07-27 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/188#discussion_r12333 --- Diff: extras/indexing/src/main/java/org/apache/rya/api/client/accumulo/AccumuloExecuteSparqlQuery.java --- @@ -0,0 +1,150

[GitHub] incubator-rya pull request #188: RYA-318 Rya Shell improvements for packagin...

2017-07-27 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/188#discussion_r129998856 --- Diff: extras/indexing/src/main/java/org/apache/rya/api/client/accumulo/AccumuloExecuteSparqlQuery.java --- @@ -0,0 +1,150

[GitHub] incubator-rya pull request #188: RYA-318 Rya Shell improvements for packagin...

2017-07-27 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/188#discussion_r12058 --- Diff: extras/indexing/src/main/java/org/apache/rya/api/client/accumulo/AccumuloExecuteSparqlQuery.java --- @@ -0,0 +1,150

[GitHub] incubator-rya issue #183: RYA-323 Enabling Multi Var Geo Functions. Closes #...

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/183 @pujav65 yeah puja hold off on merging any of these PRs until we get the build sorted. we'll test out @isper3at's #187 and #186 PRs first, then determine if need this PR. --- If your

[GitHub] incubator-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 asfbot build --- 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-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 asfbot build --- 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-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 asfbot build --- 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-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 asfbot build --- 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-rya issue #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/187 asfbot build --- 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-rya pull request #187: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/187 Rya 317 Added Near function for geo mongodb (Note this is a continuation of PR #186. This PR includes additional commits that will be squashed) ## Description >What Chan

[GitHub] incubator-rya issue #186: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/186 @isper3at also: bonus for the use of "spheroid complement disc" :1st_place_medal: --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-rya issue #186: Rya 317 Added Near function for geo mongodb

2017-07-27 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/186 @isper3at this commit looks good. I don't have any comments on my initial pass. I'm building and testing this locally right now to see if I have any issues with it. @meiercaleb

[GitHub] incubator-rya pull request #177: RYA-280-Periodic Query Service

2017-07-26 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/177#discussion_r129669294 --- Diff: extras/rya.indexing.pcj/src/main/java/org/apache/rya/indexing/pcj/storage/PeriodicQueryStorageMetadata.java --- @@ -0,0 +1,99

[GitHub] incubator-rya pull request #177: RYA-280-Periodic Query Service

2017-07-26 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/177#discussion_r129670570 --- Diff: extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/accumulo/AccumuloPeriodicQueryResultStorageIT.java

[GitHub] incubator-rya pull request #177: RYA-280-Periodic Query Service

2017-07-26 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/177#discussion_r129668480 --- Diff: extras/rya.indexing.pcj/src/main/java/org/apache/rya/indexing/pcj/storage/accumulo/PeriodicQueryTableNameFactory.java --- @@ -0,0 +1,55

[GitHub] incubator-rya issue #179: RYA-309 Added delete support to indexers

2017-07-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/179 @meiercaleb is good to go. --- 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-rya issue #179: RYA-309 Added delete support to indexers

2017-07-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/179 asfbot build --- 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-rya issue #183: RYA-323 Enabling Multi Var Geo Functions. Closes #...

2017-07-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/183 @isper3at I can't wait to see your PR We might still need my PR to do a multi-var geo:equals. Although I wouldn't doubt that you fixed that as well. :+1: --- If your project

[GitHub] incubator-rya issue #183: RYA-323 Enabling Multi Var Geo Functions. Closes #...

2017-07-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/183 @isper3at Can you look at this too. This PR is required to enable one of the main use cases for the geo:nearby function that you're working on --- If your project is set up for it, you can

[GitHub] incubator-rya pull request #183: RYA-323 Enabling Multi Var Geo Functions. C...

2017-07-25 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/183 RYA-323 Enabling Multi Var Geo Functions. Closes #183 ## Description >What Changed? [Brief Description of what changed] ### Tests >Coverage? [Descr

[GitHub] incubator-rya pull request #181: Rya-307 MongoDB Rya DAO Batch Writer

2017-07-24 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/181#discussion_r129044540 --- Diff: extras/indexing/src/main/java/org/apache/rya/indexing/mongodb/AbstractMongoIndexer.java --- @@ -68,15 +73,28 @@ protected

[GitHub] incubator-rya issue #181: Rya-307 MongoDB Rya DAO Batch Writer

2017-07-24 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/181 asfbot build --- 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-rya issue #179: RYA-309 Added delete support to indexers

2017-07-19 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/179 asfbot build --- 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-rya pull request #179: RYA-309 Added delete support to indexers

2017-07-19 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/179#discussion_r128336316 --- Diff: extras/rya.geoindexing/src/test/java/org/apache/rya/indexing/mongo/MongoGeoIT.java --- @@ -0,0 +1,135 @@ +package

[GitHub] incubator-rya pull request #179: RYA-309 Added delete support to indexers

2017-07-19 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/179#discussion_r128336188 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBRyaDAO.java --- @@ -93,7 +93,7 @@ public MongoDBRyaDAO(final

[GitHub] incubator-rya pull request #179: RYA-309 Added delete support to indexers

2017-07-19 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/179#discussion_r128335780 --- Diff: extras/rya.geoindexing/src/test/java/org/apache/rya/indexing/mongo/MongoGeoIT.java --- @@ -0,0 +1,135 @@ +package

[GitHub] incubator-rya pull request #176: Rya 268

2017-07-18 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/176 --- 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-rya issue #178: RYA-315 Fixing issue with Core Indices on MongoDB

2017-07-18 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/178 asfbot build --- 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-rya pull request #175: Rya 268 Adding in Geo Predicate Filter List

2017-07-13 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/175 --- 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-rya pull request #161: RYA-273-Construct Query Support

2017-06-21 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/161#discussion_r123325067 --- Diff: extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/query/ConstructQueryMetadata.java --- @@ -0,0 +1,192

[GitHub] incubator-rya pull request #161: RYA-273-Construct Query Support

2017-06-21 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/161#discussion_r123325785 --- Diff: extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/query/ConstructQueryMetadata.java --- @@ -0,0 +1,192

[GitHub] incubator-rya issue #138: Rya 236 - GeoTemporal Indexing

2017-06-14 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/138 Wow. These failures aren't really failures. You just so happened to get the same flaky machine (H22) three times in a row. I rebuilt your changes [here ](https://builds.apache.org/job

[GitHub] incubator-rya issue #171: RYA-288: corrected path to launch PcjAdminClient

2017-06-14 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/171 asfbot build --- 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-rya issue #171: RYA-288: corrected path to launch PcjAdminClient

2017-06-14 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/171 afsbot build --- 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-rya pull request #166: RYA-268 Adding Geoindexing Profile to Web p...

2017-06-12 Thread amihalik
GitHub user amihalik reopened a pull request: https://github.com/apache/incubator-rya/pull/166 RYA-268 Adding Geoindexing Profile to Web project ## Description Added a geoindexing profile to the web project ### Tests no new tests ### Links [Jira RYA-268

[GitHub] incubator-rya issue #166: RYA-268 Adding Geoindexing Profile to Web project

2017-06-12 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/166 asfbot build --- 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-rya pull request #166: RYA-268 Adding Geoindexing Profile to Web p...

2017-06-12 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/166 --- 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-rya issue #170: RYA-284: updated to support the use of bloom filte...

2017-06-12 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/170 asfbot build --- 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-rya issue #170: RYA-284: updated to support the use of bloom filte...

2017-06-12 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/170 I'm seeing some new tests errors and failures when I try to build this PR on my machine. I'll have asfbot build this on b.a.o. --- If your project is set up for it, you can reply

[GitHub] incubator-rya pull request #169: RYA-284: updated to support bloom filter

2017-06-08 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/169#discussion_r120970204 --- Diff: dao/accumulo.rya/src/main/java/org/apache/rya/accumulo/query/AccumuloRyaQueryEngine.java --- @@ -374,56 +387,64 @@ protected String

[GitHub] incubator-rya pull request #168: Pr jenkins test2

2017-06-02 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/168 --- 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-rya pull request #167: Testing Jenkins PR Build... Ignore this PR.

2017-06-02 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/167 --- 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-rya issue #168: Pr jenkins test2

2017-06-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/168 asfbot build --- 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-rya issue #166: RYA-268 Adding Geoindexing Profile to Web project

2017-06-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/166 asfbot build --- 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-rya issue #167: Testing Jenkins PR Build... Ignore this PR.

2017-06-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/167 asfbot build --- 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-rya issue #168: Pr jenkins test2

2017-06-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/168 asfbot build --- 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-rya issue #166: RYA-268 Adding Geoindexing Profile to Web project

2017-06-02 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/166 asfbot build --- 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-rya pull request #168: Pr jenkins test2

2017-06-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/168 Pr jenkins test2 ## Description >What Changed? [Brief Description of what changed] ### Tests >Coverage? [Description of what tests were w

[GitHub] incubator-rya pull request #167: Testing Jenkins PR Build... Ignore this PR.

2017-06-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/167 Testing Jenkins PR Build... Ignore this PR. Testing Jenkins PR Build... Ignore this PR. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-rya pull request #166: RYA-268 Adding Geoindexing Profile to Web p...

2017-06-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/166 RYA-268 Adding Geoindexing Profile to Web project ## Description Added a geoindexing profile to the web project ### Tests no new tests ### Links [Jira RYA-268

[GitHub] incubator-rya issue #151: RYA-253 Converted the rya.prospector class from Gr...

2017-04-12 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/151 asfbot build --- 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-rya issue #152: RYA-253 Build Fix for Geowave repo move

2017-04-12 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/152 @meiercaleb yep, I built it locally after blowing away geowave in my .m2 repo --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-rya pull request #152: RYA-253 Build Fix for Geowave repo move

2017-04-11 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/152 RYA-253 Build Fix for Geowave repo move ## Description >What Changed? [Brief Description of what changed] ### Tests >Coverage? [Description of what

[GitHub] incubator-rya pull request #139: RYA-241 Adding GeoWave indexer to rya.geoin...

2017-02-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/139#discussion_r99417564 --- Diff: extras/rya.geoindexing/src/main/java/org/apache/rya/indexing/accumulo/geo/GeoWaveGeoIndexer.java --- @@ -0,0 +1,661

[GitHub] incubator-rya pull request #139: RYA-241 Adding GeoWave indexer to rya.geoin...

2017-02-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/139#discussion_r99424446 --- Diff: extras/indexingExample/src/main/assembly/assembly.xml --- @@ -33,6 +33,7 @@ under the License

[GitHub] incubator-rya pull request #139: RYA-241 Adding GeoWave indexer to rya.geoin...

2017-02-03 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/139#discussion_r99424572 --- Diff: pom.xml --- @@ -80,6 +80,7 @@ under the License. 0.9.2 1.3.0-m1 +0.9.4-SNAPSHOT --- End

[GitHub] incubator-rya pull request #138: [WIP] Rya 236 - GeoTemporal Indexing

2017-01-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/138#discussion_r96710955 --- Diff: extras/rya.geotemporalindexing/src/test/java/org/apache/rya/indexing/GeoTemporalIndexerTest.java --- @@ -0,0 +1,945 @@ +package

[GitHub] incubator-rya pull request #138: [WIP] Rya 236 - GeoTemporal Indexing

2017-01-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/138#discussion_r96703892 --- Diff: extras/rya.geotemporalindexing/src/test/java/org/apache/rya/indexing/GeoTemporalIndexerTest.java --- @@ -0,0 +1,945 @@ +package

  1   2   >