You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rya.apache.org by pujav65 <gi...@git.apache.org> on 2015/12/17 05:29:36 UTC
[GitHub] incubator-rya pull request: RYA-17, RYA-19, RYA-20 issue with mong...
GitHub user pujav65 opened a pull request:
https://github.com/apache/incubator-rya/pull/6
RYA-17, RYA-19, RYA-20 issue with mongo deletes, typo in constructor,…
… redundant indices
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pujav65/incubator-rya RYA-17
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-rya/pull/6.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 #6
----
commit 288c7742ec7fd12e29b94092708742aab8e104f8
Author: pujav65 <pu...@gmail.com>
Date: 2015-12-17T04:23:25Z
RYA-17, RYA-19, RYA-20 issue with mongo deletes, typo in constructor, redundant indices
----
---
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-17, RYA-19, RYA-20, RYA-10 issue w...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-166724892
Yup. I missed what was happening with jira today -- everything in a good state? Want me to look at it at all?
Sent from my iPhone
> On Dec 22, 2015, at 10:45 AM, Aaron Mihalik <no...@github.com> wrote:
>
> Does this PR close all of these issues? RYA-17, RYA-19, RYA-20, RYA-10?
>
> —
> Reply to this email directly or view it on GitHub.
>
---
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-17, RYA-19, RYA-20, RYA-10 issue w...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 closed the pull request at:
https://github.com/apache/incubator-rya/pull/6
---
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-17, RYA-19, RYA-20, RYA-10 issue w...
Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-166931690
merged into develop
---
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-17, RYA-19, RYA-20 issue with mong...
Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-165796392
Looks good. I'll pull it in.
---
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-17, RYA-19, RYA-20 issue with mong...
Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-165798435
You added a new file (`MongoDBRyaDAOTest.java`) and it doesn't have a license header (so it failed `mvn apache-rat:check`). Please go back in and add the header, then I'll pull in this PR.
---
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-17, RYA-19, RYA-20 issue with mong...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-166493257
fixed the license on the new test, and then implemented something for RYA-10 (reusing existing collections for geo queries). Someone should implement more of the geo functionality, that can be a follow on task though.
---
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-17, RYA-19, RYA-20, RYA-10 issue w...
Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-166651492
Does this PR close all of these issues? RYA-17, RYA-19, RYA-20, RYA-10?
---
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-17, RYA-19, RYA-20, RYA-10 issue w...
Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the pull request:
https://github.com/apache/incubator-rya/pull/6#issuecomment-168749451
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 the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---