afs 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_r319648994
##########
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:
Yes. While that is the default implementation, it is the one used by TDB2.
Consider whether the app wants the right answer during a transaction ...
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
