[jira] [Commented] (RYA-73) Add insert time to mongo backed rya

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-73?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15313053#comment-15313053
 ] 

ASF GitHub Bot commented on RYA-73:
---

Github user pujav65 commented on the issue:

https://github.com/apache/incubator-rya/pull/43
  
looks good.  will merge tomorrow.  good job adding tests.


> Add insert time to mongo backed rya
> ---
>
> Key: RYA-73
> URL: https://issues.apache.org/jira/browse/RYA-73
> Project: Rya
>  Issue Type: Improvement
>Reporter: Andrew Smith
>
> Some features of accumulo backed Rya require db insert time.  This is not 
> given for free in mongo.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (RYA-73) Add insert time to mongo backed rya

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-73?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15313054#comment-15313054
 ] 

ASF GitHub Bot commented on RYA-73:
---

Github user pujav65 commented on the issue:

https://github.com/apache/incubator-rya/pull/43
  
oh kind of not related to this pr, but should we consider adding context to 
the mongo composite indices?


> Add insert time to mongo backed rya
> ---
>
> Key: RYA-73
> URL: https://issues.apache.org/jira/browse/RYA-73
> Project: Rya
>  Issue Type: Improvement
>Reporter: Andrew Smith
>
> Some features of accumulo backed Rya require db insert time.  This is not 
> given for free in mongo.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (RYA-73) Add insert time to mongo backed rya

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-73?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15287064#comment-15287064
 ] 

ASF GitHub Bot commented on RYA-73:
---

GitHub user isper3at opened a pull request:

https://github.com/apache/incubator-rya/pull/43

RYA-73 added timestamp to mongoDB DAO

Changed the mongo storage strategy to now
include timestamp in the [de]serialization
of RyaStatements.

The timestamp is added at insertion timein the DAO

Added test for [de]serialize
Added test for timestamp in the DAO
Added failsafe to mongo rya for integration test framework

[Jira](https://issues.apache.org/jira/browse/RYA-73)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/isper3at/incubator-rya mongoTimestamp

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-rya/pull/43.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 #43


commit f6d2db3557c093dde24f445711bec4211824233b
Author: isper3at 
Date:   2016-05-09T18:29:18Z

RYA-73 added timestamp to mongoDB DAO

Changed the mongo storage strategy to now
include timestamp in the [de]serialization
of RyaStatements.

The timestamp is added at insertion timein the DAO

Added test for [de]serialize
Added test for timestamp in the DAO




> Add insert time to mongo backed rya
> ---
>
> Key: RYA-73
> URL: https://issues.apache.org/jira/browse/RYA-73
> Project: Rya
>  Issue Type: Improvement
>Reporter: Andrew Smith
>
> Some features of accumulo backed Rya require db insert time.  This is not 
> given for free in mongo.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)