[GitHub] [jena] ajs6f commented on a change in pull request #598: JENA-1748: Tests and fix for TDB2 DatasetGraph listGraphNodes

[GitBox <gi...@apache.org>]

ajs6f commented on a change in pull request #598: JENA-1748: Tests and fix for TDB2 DatasetGraph listGraphNodes
URL: https://github.com/apache/jena/pull/598#discussion_r319529442
 
 

 ##########
 File path: jena-db/jena-dboe-storage/src/main/java/org/apache/jena/dboe/storage/system/DatasetGraphStorage.java
 ##########
 @@ -83,7 +83,7 @@ public DatasetGraphStorage(StorageRDF storage, StoragePrefixes prefixes, Transac
 
     @Override
     public Iterator<Node> listGraphNodes() {
-        Iterator<Quad> iter = find(null, null, null, null);
+        Iterator<Quad> iter = findNG(null, null, null, null);
 
 Review comment:
   Just a question for my learning: am I reading this correctly that TDB2 gets its list of graphs by iterating every quad in the store?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services