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

    https://github.com/apache/jena/pull/474#discussion_r221716057
  
    --- Diff: 
jena-elephas/jena-elephas-mapreduce/src/main/java/org/apache/jena/hadoop/rdf/mapreduce/filter/AbstractNodeTupleFilterMapper.java
 ---
    @@ -28,9 +28,7 @@
     /**
      * Abstract mapper implementation which helps in filtering tuples from the
      * input, derived implementations provide an implementation of the
    - * {@link #accepts(TKey, T)}
    - * 
    - * 
    + * {@link #accepts(Object, AbstractNodeTupleWritable)}
    --- End diff --
    
    This just looks wrong-- the method below clearly is `accepts(TKey key, T 
tuple)`, and the concrete class isn't there at all.


---

Reply via email to