[
https://issues.apache.org/jira/browse/RYA-53?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15235190#comment-15235190
]
ASF GitHub Bot commented on RYA-53:
-----------------------------------
Github user kchilton2 commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/36#discussion_r59215674
--- Diff:
extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/JoinResultUpdater.java
---
@@ -407,15 +411,35 @@ public boolean hasNext() {
public BindingSet next() {
final MapBindingSet bs = new MapBindingSet();
- for(Binding binding : newResult) {
+ for(final Binding binding : newResult) {
bs.addBinding(binding);
}
- for(Binding binding : joinedResults.next()) {
+ final BindingSet joinResult = joinedResults.next();
+ for(final Binding binding : joinResult) {
bs.addBinding(binding);
}
- return bs;
+ String visibility = "";
+ if(newResult instanceof VisibilityBindingSet && joinResult
instanceof VisibilityBindingSet) {
--- End diff --
This is another area where we should be able to assume we're using
VisibilityBindingSets.
> 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)
