Github user isper3at commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/251#discussion_r153007455
--- Diff:
extras/rya.pcj.fluo/pcj.fluo.api/src/main/java/org/apache/rya/indexing/pcj/fluo/api/GetQueryReport.java
---
@@ -132,14 +135,20 @@ private BigInteger countBindingSets(final
SnapshotBase sx, final String nodeId,
checkNotNull(nodeId);
checkNotNull(bindingSetColumn);
+ NodeType type = NodeType.fromNodeId(nodeId).get();
+ Bytes prefixBytes = Bytes.of(type.getNodeTypePrefix());
+
// Limit the scan to the binding set column and node id.
- final RowScanner rows =
sx.scanner().over(Span.prefix(nodeId)).fetch(bindingSetColumn).byRow().build();
--- End diff --
is this changing the actual query? using nodeID vs prefixBytes. Will this
break an existing fluo instance?
---
