[jira] [Commented] (RYA-53) Add Visibility to Rya PCJs

[ASF GitHub Bot (JIRA)]

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

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

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

    https://github.com/apache/incubator-rya/pull/36#discussion_r58912007
  
    --- Diff: 
extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/observers/TripleObserver.java
 ---
    @@ -85,9 +97,16 @@ public void process(final TypedTransactionBase tx, final 
Bytes row, final Column
     
                 while(ci.hasNext()) {
                     final String varOrders = ci.next().getValue().toString();
    -                final String bindingSet = getBindingSet(triple, pattern, 
varOrders);
    -                if(bindingSet.length() != 0) {
    -                    tx.mutate().row(spID + NODEID_BS_DELIM + 
bindingSet).col(FluoQueryColumns.STATEMENT_PATTERN_BINDING_SET).set(bindingSet);
    +                final VariableOrder varOrder = new 
VariableOrder(varOrders);
    +                final String bindingSetString = getBindingSet(triple, 
pattern, varOrders);
    +
    --- End diff --
    
    whitespace


> Add Visibility to Rya PCJs
> --------------------------
>
>                 Key: RYA-53
>                 URL: https://issues.apache.org/jira/browse/RYA-53
>             Project: Rya
>          Issue Type: New Feature
>            Reporter: Andrew Smith
>
> Update the PCJ api to support visibility attached to binding sets



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