Github user ajs6f commented on a diff in the pull request:

    https://github.com/apache/jena/pull/434#discussion_r194269455
  
    --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/core/GraphView.java 
---
    @@ -165,6 +165,9 @@ public void performDelete( Triple t ) {
             Node o = t.getObject() ;
             dsg.delete(g, s, p, o) ;
         }
    +    
    +    // Subclasses may wish to provide graphBaseSize otherwise GraphBase 
uses find()  
    --- End diff --
    
    Maybe this is just where I like to find things, but I'd rather `@Override 
protected int graphBaseSize() { return super.graphBaseSize(); }` with this as a 
Javadoc comment. 


---

Reply via email to