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

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

Github user isper3at commented on a diff in the pull request:

    https://github.com/apache/incubator-rya/pull/124#discussion_r89895899
  
    --- Diff: 
dao/mongodb.rya/src/main/java/org/apache/rya/mongodb/MongoDBRyaDAO.java ---
    @@ -146,15 +153,18 @@ public void destroy() throws RyaDAOException {
         public void add(final RyaStatement statement) throws RyaDAOException {
             // add it to the collection
             try {
    -            coll.insert(storageStrategy.serialize(statement));
    -            for(final RyaSecondaryIndexer index: secondaryIndexers) {
    -                index.storeStatement(statement);
    +            final boolean canAdd = 
DocumentVisibilityUtil.doesUserHaveDocumentAccess(auths, 
statement.getColumnVisibility());
    --- End diff --
    
    I'll check later, but should this be doesUserHaveCollectionAccess?


> Add visibility support to MongoDB
> ---------------------------------
>
>                 Key: RYA-119
>                 URL: https://issues.apache.org/jira/browse/RYA-119
>             Project: Rya
>          Issue Type: Improvement
>          Components: dao
>    Affects Versions: 3.2.10
>            Reporter: Andrew Smith
>            Assignee: Eric White
>
> Currently, when querying mongo, visibility is ignored.  Need to add support 
> for visibility when querying mongo.



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

Reply via email to