[GitHub] incubator-rya pull request: RYA-35 - Added a class that converts B...

2016-02-18 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/18#issuecomment-185722647 Looks good. Thanks! --- 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] incubator-rya pull request: RYA-41 Fixed a bug where AccumuloIndex...

2016-02-19 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/20#issuecomment-186440469 Looks good. I'll merge this on Sunday unless there are other concerns. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-rya pull request: RYA-37 Bump GeoMesa version to 1.2.0

2016-02-22 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/19#issuecomment-187341081 Thanks! Congrats on the release! --- 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

[GitHub] incubator-rya pull request: Applying Correct License

2016-03-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/24 Applying Correct License You can merge this pull request into a Git repository by running: $ git pull https://github.com/amihalik/incubator-rya develop Alternatively you can review

[GitHub] incubator-rya pull request: Applying Correct License

2016-03-02 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/24 --- 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] incubator-rya pull request: RYA-7 Applying Correct License

2016-03-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/25 RYA-7 Applying Correct License You can merge this pull request into a Git repository by running: $ git pull https://github.com/amihalik/incubator-rya develop Alternatively you can

[GitHub] incubator-rya pull request: RYA-7 Applying Correct License

2016-03-03 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/25#issuecomment-191694535 Thanks @joshelser, but I need a bit more help. I don't quite understand the apache-rat behavior. We have those exclusions in the parent pom, we

[GitHub] incubator-rya pull request: RYA-7 Applying Correct License

2016-03-03 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/25#issuecomment-192046091 gotcha, thanks! I'll put in another issue to set the configuration in each sub-module. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-rya pull request: RYA-7 Applying Correct License

2016-03-03 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/25 --- 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] incubator-rya pull request: RYA-45 Pushing apache-rat constraints ...

2016-03-03 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/26 RYA-45 Pushing apache-rat constraints to submodules You can merge this pull request into a Git repository by running: $ git pull https://github.com/amihalik/incubator-rya RYA-45

[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...

2016-03-04 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/21#issuecomment-192227049 If you fix the conflicts below, I'll merge your PR: ``` CONFLICT (content): Merge conflict in extras/indexingExample/src/main

[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...

2016-03-14 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/21#issuecomment-196482681 committed. --- 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

[GitHub] incubator-rya pull request: updated Accumulo and Hadoop version nu...

2016-03-25 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/34#issuecomment-201605097 Thanks! I merged this PR. Please close 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

[GitHub] incubator-rya pull request: RYA-50 Updated Rya PCJs to support SPA...

2016-04-04 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/35#discussion_r58443882 --- Diff: extras/indexing/src/main/java/mvm/rya/indexing/external/tupleSet/AccumuloPcjSerializer.java --- @@ -19,104 +19,198 @@ * under the

[GitHub] incubator-rya pull request: Input format fix

2016-04-06 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/32#issuecomment-206569001 Puja: I updated the committer instructions on the [confluence](https://cwiki.apache.org/confluence/display/RYA/Contributing+to+Rya) page. I used this PR as the

[GitHub] incubator-rya pull request: Input format fix

2016-04-07 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/32#issuecomment-207025396 @jej2003 the exceptions are expected. These tests work fine on my box, so I'm fine if @pujav65 merges it. --- If your project is set up for it, you can

[GitHub] incubator-rya pull request: Rya 128 add admin tables

2016-05-18 Thread amihalik
Github user amihalik commented on the pull request: https://github.com/apache/incubator-rya/pull/44#issuecomment-220173471 Is this complete? --- 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

[GitHub] incubator-rya pull request #46: Adding Visibilties to Rya Web

2016-06-02 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/46 Adding Visibilties to Rya Web Modified the Rya Web Rest service to take in visibilites; renamed integration test to run as unit test; updated Example to use visibilities and auths

[GitHub] incubator-rya pull request #51: Upgrading versions of geomesa

2016-06-23 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/51#discussion_r68250007 --- Diff: dao/accumulo.rya/src/main/java/mvm/rya/accumulo/AccumuloRyaDAO.java --- @@ -202,12 +202,18 @@ public void delete(Iterator statements

[GitHub] incubator-rya issue #56: Add Merge Tool

2016-07-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/56 Did you mean to check in those accumulo table files (i.e. /test/copy_tool_file_output/**)? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-rya pull request #57: RYA-130 Fixed Rya Mongo Multi Binding Set Jo...

2016-07-08 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/57 RYA-130 Fixed Rya Mongo Multi Binding Set Join Fixed bug in Mongo Query Engine; added test You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-rya pull request #65: RYA-148 Fixed a bug where Joins would write ...

2016-08-11 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/65#discussion_r74498673 --- Diff: extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/JoinResultUpdater.java --- @@ -420,10 +423,20 @@ public

[GitHub] incubator-rya pull request #67: Rya 101 and 102 Mongo Rya example fails to i...

2016-08-17 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/67#discussion_r75194741 --- Diff: dao/mongodb.rya/src/main/java/mvm/rya/mongodb/MongoConnectorFactory.java --- @@ -41,26 +47,77 @@ * @param conf The {@link

[GitHub] incubator-rya pull request #46: Adding Visibilties to Rya Web

2016-08-18 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/46 --- 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] incubator-rya pull request #57: RYA-130 Fixed Rya Mongo Multi Binding Set Jo...

2016-08-18 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/57 --- 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] incubator-rya pull request #68: [WIP] RYA-151 Rya Query benchmark tool imple...

2016-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/68#discussion_r75303476 --- Diff: extras/rya.benchmark/src/test/java/org/apache/rya/benchmark/query/BenchmarkQueriesReaderIT.java --- @@ -0,0 +1,55

[GitHub] incubator-rya pull request #69: RYA-149 Updating pom to make the release eas...

2016-08-18 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/69 RYA-149 Updating pom to make the release easier You can merge this pull request into a Git repository by running: $ git pull https://github.com/amihalik/incubator-rya RYA-149

[GitHub] incubator-rya issue #70: [WIP] RYA-109 Add Spark Support

2016-08-18 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/70 Something is screwy with Jesse's commit that you pulled in. The commit you have has hash 1855213 and the one in the apache/develop branch has 01489ef. This needs to be fixed before w

[GitHub] incubator-rya pull request #68: [WIP] RYA-151 Rya Query benchmark tool imple...

2016-08-18 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/68#discussion_r75355985 --- Diff: extras/rya.benchmark/src/test/java/org/apache/rya/benchmark/query/BenchmarkQueriesReaderIT.java --- @@ -0,0 +1,55

[GitHub] incubator-rya issue #72: RYA-133 Fix broken Mongo Secondary Indexers

2016-08-18 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/72 No comments on your PR commit, but I really like your description. I think we should do this: [RYA-156 - Add a Pull Request Template](https://issues.apache.org/jira/browse/RYA-156) --- If

[GitHub] incubator-rya issue #28: Update apache licenses + add servlet-api dependency

2016-08-19 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/28 I'd hold off a bit on merging this. It upgrades some dependencies. Wait for the release. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-rya issue #56: Add Merge Tool

2016-08-19 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/56 @ejwhite922 going back to the binary files like: extras/rya.merger/resources/test/copy_tool_file_output/ct_spo/files/part-r-5.rf I'm concerned that your tests bas

[GitHub] incubator-rya pull request #69: RYA-149 Updating pom to make the release eas...

2016-08-23 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/69 --- 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] incubator-rya issue #54: RYA-100 Added Maven animal-sniffer plugin to build.

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/54 Merged. --- 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

[GitHub] incubator-rya issue #60: Rya 81

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/60 Merged. --- 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

[GitHub] incubator-rya issue #65: RYA-148 Fixed a bug where Joins would write the sam...

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/65 Merged. --- 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

[GitHub] incubator-rya issue #64: RYA-143 Updated the Mongo DB Java Driver to 3.3.0.

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/64 Merged. --- 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

[GitHub] incubator-rya issue #63: RYA-103 Updates the Rya build to use Java 1.8

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/63 Merged. --- 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

[GitHub] incubator-rya issue #62: Rya 82

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/62 Merged. --- 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

[GitHub] incubator-rya issue #76: RYA-156 Added Pull request template

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/76 Merged. --- 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

[GitHub] incubator-rya issue #72: RYA-133 Fix broken Mongo Secondary Indexers

2016-08-23 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/72 Merged. --- 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

[GitHub] incubator-rya issue #79: RYA-54 vagrant cleanup

2016-08-24 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/79 looks good. merged. --- 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

[GitHub] incubator-rya issue #80: RYA-163 PCJDetails Test Failures on non-EST TimeZon...

2016-08-24 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/80 Nice... super simple. I tested it out and it passed on my machine. --- 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-rya pull request #80: RYA-163 PCJDetails Test Failures on non-EST ...

2016-08-24 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/80#discussion_r76156372 --- Diff: extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java --- @@ -123,6 +124,9 @@ public void deletePCJ() throws

[GitHub] incubator-rya pull request #80: RYA-163 PCJDetails Test Failures on non-EST ...

2016-08-24 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/80#discussion_r76162747 --- Diff: extras/rya.console/src/test/java/mvm/rya/shell/RyaAdminCommandsTest.java --- @@ -123,6 +124,9 @@ public void deletePCJ() throws

[GitHub] incubator-rya issue #81: [WIP] RYA-168: property chain inference support

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/81 You need to fix your whitespace issues. Set up eclipse to use spaces, not tabs. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-rya issue #51: Upgrading versions of geomesa

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/51 Please change this to point to the apache master branch. Also, what's the status of this PR? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-rya issue #59: Adding a dynamo db backend for Rya

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/59 Please change this to point to the apache master branch. Also, what's the status of this PR? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-rya issue #66: [WIP] Rya 147 - PCJ Delete Statement Support

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/66 Please change this to point to the apache master branch. --- 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 #73: [WIP] RYA-94 Remove from secondary indexes when del...

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/73 Please change this to point to the apache master branch. --- 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 #71: [WIP] RYA-109 Feature/spark graph x

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/71 Please change this to point to the apache master branch. --- 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 #68: [WIP] RYA-151 Rya Query benchmark tool implemented ...

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/68 Please change this to point to the apache master branch. --- 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 #75: [WIP] Rya 123 Refactored Merge Tool

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/75 Please change this to point to the apache master branch. --- 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 #77: RYA-153 Fixed JoinPCJMatcher bug

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/77 Please change this to point to the apache master branch. --- 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 #77: RYA-153 Fixed JoinPCJMatcher bug

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/77 Please change this to point to the apache master branch. --- 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 #81: [WIP] RYA-168: property chain inference support

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/81 Please change this to point to the apache master branch. --- 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 #78: RYA-154 Decoupled Updater Configuration from PCJ Co...

2016-08-25 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/78 Please change this to point to the apache master branch. --- 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 #88: RYA-173 Fixed signing of rya.merger distribution zi...

2016-09-08 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/88 Looks good. I'm going to test it out. If it works, I'll pull it in and cut an RC2 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-rya pull request #90: RYA-165 Simplify PCJ result visibility expre...

2016-09-13 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/90#discussion_r78598233 --- Diff: dao/accumulo.rya/src/main/java/mvm/rya/accumulo/utils/VisibilitySimplifier.java --- @@ -0,0 +1,54 @@ +/* + * Licensed to the Apache

[GitHub] incubator-rya pull request #93: RYA-176 Adding DEPENDENCIES to RAT Exclusion

2016-09-13 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/93 RYA-176 Adding DEPENDENCIES to RAT Exclusion ## Description Updated the POMs to ignore `DEPENDENCIES` file contained in release. Also some minor updates (`rya.console` had a log file

[GitHub] incubator-rya pull request #85: RYA-151 - Benchmarking Tools

2016-09-13 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/85#discussion_r78671518 --- Diff: extras/indexing/src/main/java/mvm/rya/indexing/external/PrecomputedJoinStorageSupplier.java --- @@ -27,6 +27,7 @@ import

[GitHub] incubator-rya pull request #85: RYA-151 - Benchmarking Tools

2016-09-13 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/85#discussion_r78671763 --- Diff: extras/rya.benchmark/pom.xml --- @@ -0,0 +1,250 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] incubator-rya pull request #85: RYA-151 - Benchmarking Tools

2016-09-13 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/85#discussion_r78671939 --- Diff: extras/rya.benchmark/pom.xml --- @@ -0,0 +1,250 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] incubator-rya pull request #85: RYA-151 - Benchmarking Tools

2016-09-14 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/85#discussion_r78775502 --- Diff: extras/rya.benchmark/pom.xml --- @@ -0,0 +1,250 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] incubator-rya issue #99: rya-177 WIP; indexing and mapreduce are now optiona...

2016-10-06 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/99 A couple things: I cannot do a `mvn clean install` . Apache rat breaks. You need to remove more projects including: - tinkerpop.rya - web.rya - rya.reasoning

[GitHub] incubator-rya issue #101: [WIP] RYA-177 adding optional geoindexing profile

2016-10-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/101 I'm still seeing gt libs pulled into the build eg: ``` [INFO] [INFO] Building Apache Rya Seco

[GitHub] incubator-rya issue #101: [WIP] RYA-177 adding optional geoindexing profile

2016-10-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/101 Better. I'm still seeing gt dependencies. try running `mvn dependency:tree | grep gt` in your environment. I get this: ``` [INFO] | | | +- org.geotoo

[GitHub] incubator-rya issue #101: [WIP] RYA-177 adding optional geoindexing profile

2016-10-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/101 Yep, I will missing a commit. Looks good. Building and running the tests now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-rya pull request #102: RYA-187 Added 'Incubating' to artifact name...

2016-10-12 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/102 RYA-187 Added 'Incubating' to artifact names ## Description Changed version numbering from 3.2.10-SNAPSHOT to 3.2.10-incubating-SNAPSHOT ### Tests N

[GitHub] incubator-rya pull request #103: RYA-201 Renaming Geospatial Indexing projec...

2016-10-13 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/103 RYA-201 Renaming Geospatial Indexing project ## Description >What Changed? Project name in POM file ### Tests ### Links [Jira](https://issues.apache.

[GitHub] incubator-rya pull request #111: Rya 180

2016-10-14 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/111 Rya 180 ## Description Upgraded to Apache Tinkerpop and Removed tinkerpop.rya in order to remove LGPL dependencies (including HEP Dependency) ### Tests Ran current tests

[GitHub] incubator-rya issue #107: RYA-178-Remove Rat Exclusions for OWL Files

2016-10-14 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/107 delete `sail/src/test/resources/univ-bench.owl` completely. It appears that we don't use it in any tests. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-rya pull request #113: Rya 198

2016-10-14 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/113 Rya 198 ## Description Change package names to org.apache. ### Tests Ran existing tests ### Links [Jira](https://issues.apache.org/jira/browse/RYA-198

[GitHub] incubator-rya issue #110: Rya infra push

2016-10-16 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/110 Yahiaoui, The best way to contact us is through the Apache Rya mailing list. Send an email to dev-subscr...@rya.incubator.apache.org to join, then send your questions to d

[GitHub] incubator-rya pull request #115: RYA-200 Removing findbugs:jsr305 Dependency

2016-10-17 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/115 RYA-200 Removing findbugs:jsr305 Dependency ## Description Removed findbugs:jsr305 Dependency. Replaced with findbugs-annotations. Several annotations are not available in

[GitHub] incubator-rya pull request #116: Adding a commit to break the build

2016-10-17 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/116 Adding a commit to break the build ## Description >What Changed? [Brief Description of what changed] ### Tests >Coverage? [Description of what test

[GitHub] incubator-rya pull request #115: RYA-200 Removing findbugs:jsr305 Dependency

2016-10-17 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/115#discussion_r83737780 --- Diff: extras/rya.console/src/main/java/org/apache/rya/shell/SharedShellState.java --- @@ -23,10 +23,10 @@ import java.util.Objects

[GitHub] incubator-rya pull request #116: Adding a commit to break the build

2016-10-18 Thread amihalik
Github user amihalik closed the pull request at: https://github.com/apache/incubator-rya/pull/116 --- 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] incubator-rya pull request #117: RYA-174 Adding Mongo Cleanup in Test

2016-10-19 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/117 RYA-174 Adding Mongo Cleanup in Test ## Description Modified tests with embedded mongo. Embedded mongo now cleans up after itself. ### Tests no new tests

[GitHub] incubator-rya issue #115: RYA-200 Removing findbugs:jsr305 Dependency

2016-10-19 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/115 @joshelser thanks a lot for finding those libraries and reading the docs in detail to find the equivalent to ParametersAreNonNullByDefault. I've updated the PR with your fin

[GitHub] incubator-rya issue #117: RYA-174 Adding Mongo Cleanup in Test

2016-10-20 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/117 @dlotts Can you take a look at this? I think I finally fixed 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

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84362556 --- Diff: extras/rya.jena.sesame/pom.xml --- @@ -0,0 +1,174 @@ + + + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi=

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84360991 --- Diff: extras/rya.jena.sesame/pom.xml --- @@ -0,0 +1,174 @@ + + + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi=

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84360640 --- Diff: extras/rya.jena.sesame/NOTICE --- @@ -0,0 +1,11 @@ +Apache Jena - ARQ module +Copyright 2011, 2012 The Apache Software Foundation

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84366245 --- Diff: extras/rya.jena.sesame/Copyright.txt --- @@ -0,0 +1,25 @@ + + (c) Copyright 2009 Talis Information Ltd. + All rights

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84360591 --- Diff: extras/rya.jena.sesame/Copyright.txt --- @@ -0,0 +1,25 @@ + + (c) Copyright 2009 Talis Information Ltd. + All rights

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84362217 --- Diff: extras/rya.jena.sesame/src/test/resources/rdf_format_files/rdfxml_files/storeFront.owl --- @@ -0,0 +1,453 @@ + --- End diff

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84363108 --- Diff: extras/rya.jena.sesame/assembly.xml --- @@ -0,0 +1,67 @@ + + + + + +http://maven.apache.org/plugins/maven-assembly

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-20 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84366486 --- Diff: extras/rya.jena.sesame/NOTICE --- @@ -0,0 +1,11 @@ +Apache Jena - ARQ module +Copyright 2011, 2012 The Apache Software Foundation

[GitHub] incubator-rya pull request #118: RYA-174 Removing Mongo Test Factory from no...

2016-10-20 Thread amihalik
GitHub user amihalik opened a pull request: https://github.com/apache/incubator-rya/pull/118 RYA-174 Removing Mongo Test Factory from non-test code >What Changed? Removed Mongo Test Factory code from non-test code ### Tests no new tests ### Li

[GitHub] incubator-rya issue #118: RYA-174 Removing Mongo Test Factory from non-test ...

2016-10-21 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/118 @isper3at Can you look at this as well? --- 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

[GitHub] incubator-rya pull request #118: RYA-174 Removing Mongo Test Factory from no...

2016-10-24 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/118#discussion_r84739883 --- Diff: extras/indexingExample/src/main/java/MongoRyaDirectExample.java --- @@ -255,19 +265,29 @@ private static void

[GitHub] incubator-rya pull request #118: RYA-174 Removing Mongo Test Factory from no...

2016-10-24 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/118#discussion_r84739864 --- Diff: dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MockMongoFactory.java --- @@ -0,0 +1,97 @@ +/* + * Licensed to the Apache

[GitHub] incubator-rya issue #118: RYA-174 Removing Mongo Test Factory from non-test ...

2016-10-24 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/118 It seems like everyone is on board with this PR... I'm going to let Jenkins chomp on it for a bit and (unless I hear otherwise) pull this into master after the build. --- If your pr

[GitHub] incubator-rya pull request #106: RYA-204 Added the Rya Jena Sesame adapter w...

2016-10-25 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/106#discussion_r84932690 --- Diff: extras/rya.jena.sesame/src/test/resources/rdf_format_files/rdfxml_files/storeFront.owl --- @@ -0,0 +1,453 @@ + --- End diff

[GitHub] incubator-rya issue #91: Rya 106 - Ensure users are able to access all or no...

2016-10-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/91 Wow! Thanks @isper3at I'll put this in the queue of things to merge after the release is finalized. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-rya issue #94: RYA-110 - Add Uninstall command to Rya Console.

2016-10-26 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/94 @isper3at I just noticed your list at [RYA-208](https://issues.apache.org/jira/browse/RYA-208). I'll comment on that ticket as I merge them. Thanks again. --- If your project is s

[GitHub] incubator-rya pull request #84: Rya 123 merge tool

2016-11-01 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/84#discussion_r85941124 --- Diff: extras/rya.geoindexing/src/main/java/org/apache/rya/indexing/GeoRyaSailFactory.java --- @@ -143,4 +144,18 @@ private static AccumuloRyaDAO

[GitHub] incubator-rya pull request #84: Rya 123 merge tool

2016-11-01 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/84#discussion_r85940024 --- Diff: extras/rya.export/export.accumulo/pom.xml --- @@ -0,0 +1,131 @@ + + +http://maven.apache.org/POM/4.0.0"; +xmln

[GitHub] incubator-rya pull request #84: Rya 123 merge tool

2016-11-01 Thread amihalik
Github user amihalik commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/84#discussion_r85940183 --- Diff: extras/rya.export/export.accumulo/pom.xml --- @@ -0,0 +1,131 @@ + + +http://maven.apache.org/POM/4.0.0"; +xmln

  1   2   3   >