You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@rya.apache.org by pujav65 <gi...@git.apache.org> on 2016/02/29 17:44:27 UTC
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
GitHub user pujav65 opened a pull request:
https://github.com/apache/incubator-rya/pull/21
RYA-11 initial implementation of free text for mongo backend
Initial implementation. Mongo backed free text query does not match Accumulo, but I don't have the cycles to finish that. Updating this so that it matches would be a good future task for someone to pick up.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pujav65/incubator-rya RYA-11
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-rya/pull/21.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #21
----
commit 59073a3dcc4e2737d678255921c27e363a600b1f
Author: pujav65 <pu...@gmail.com>
Date: 2016-01-13T16:21:52Z
RYA-11 initial implementation of free text for mongo backend
----
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
Posted by amihalik <gi...@git.apache.org>.
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/java/MongoRyaDirectExample.java
CONFLICT (content): Merge conflict in extras/indexing/src/test/java/mvm/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
CONFLICT (modify/delete): extras/indexing/src/test/java/mvm/rya/indexing/external/AccumuloIndexSetTest2.java deleted in 13de8a5645391b65ba0a6753eedbb51bff4d7bee and modified
```
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the pull request:
https://github.com/apache/incubator-rya/pull/21#issuecomment-190366312
Added fixes for Rya-34 to this pull. Sorry for multiple bugs on the same PR, had to build on fixes from the original pull request
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
Posted by amihalik <gi...@git.apache.org>.
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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 closed the pull request at:
https://github.com/apache/incubator-rya/pull/21
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-rya pull request: RYA-11 initial implementation of free ...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the pull request:
https://github.com/apache/incubator-rya/pull/21#issuecomment-195375125
Hey Aaron, the merge conflict is fixed now, but I've been having trouble building. maven can't resolve the updated geomesa dependencies. Either way, I think it should be good to build, but I was going to ask you or David to check that it built (since you probably have the dependencies from an earlier build and they must have disappeared recently).
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---