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

    https://github.com/apache/jena/pull/337#discussion_r159250915
  
    --- Diff: jena-arq/src/main/java/org/apache/jena/sparql/util/Context.java 
---
    @@ -199,13 +193,12 @@ public long getLong(Symbol symbol, long defaultValue) 
{
             }
         }
         
    -    public void putAll(Context other) {
    +    public Context putAll(Context other) {
             if ( readonly )
    --- End diff --
    
    I think that was leftover from when I was still trying to find a way to 
take context from both underlying datasets. Would you rather "fluent-ify" all 
such methods?


---

Reply via email to