Github user ajs6f commented on a diff in the pull request:
https://github.com/apache/jena/pull/396#discussion_r181458378
--- Diff:
jena-db/jena-tdb2/src/main/java/org/apache/jena/tdb2/solver/stats/StatsCollectorNodeId.java
---
@@ -18,39 +18,39 @@
package org.apache.jena.tdb2.solver.stats;
-import java.util.HashMap ;
-import java.util.Map ;
+import java.util.HashMap;
+import java.util.Map;
-import org.apache.jena.graph.Node ;
-import org.apache.jena.sparql.graph.NodeConst ;
+import org.apache.jena.graph.Node;
+import org.apache.jena.sparql.graph.NodeConst;
import org.apache.jena.tdb2.store.NodeId;
import org.apache.jena.tdb2.store.nodetable.NodeTable;
/** Statistics collector, aggregates based on NodeId */
-public class StatsCollectorNodeId extends StatsCollectorBase<NodeId>
-{
- private NodeTable nodeTable ;
-
- public StatsCollectorNodeId(NodeTable nodeTable)
- {
- super(findRDFType(nodeTable)) ;
- this.nodeTable = nodeTable ;
+public class StatsCollectorNodeId extends StatsCollectorBase<NodeId> {
+ private NodeTable nodeTable;
+
+ public StatsCollectorNodeId(NodeTable nodeTable) {
+ super(findRDFType(nodeTable));
+ this.nodeTable = nodeTable;
}
-
- private static NodeId findRDFType(NodeTable nodeTable2)
- {
- return nodeTable2.getAllocateNodeId(NodeConst.nodeRDFType) ;
+
+ private static NodeId findRDFType(NodeTable nodeTable) {
+ // It may not exist.
+ NodeId nodeId = nodeTable.getNodeIdForNode(NodeConst.nodeRDFType);
+ if ( NodeId.isDoesNotExist(nodeId) )
--- End diff --
Jut for my own education-- am I right in understanding this to be the fix
for the "no type triples in the data" problem?
---
