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

    https://github.com/apache/incubator-rya/pull/251#discussion_r153265659
  
    --- Diff: 
extras/rya.pcj.fluo/pcj.fluo.app/src/main/java/org/apache/rya/indexing/pcj/fluo/app/batch/AbstractBatchBindingSetUpdater.java
 ---
    @@ -23,13 +23,17 @@
     import org.apache.fluo.api.data.RowColumn;
     import org.apache.fluo.api.data.Span;
     import org.apache.rya.indexing.pcj.fluo.app.query.FluoQueryColumns;
    +import org.apache.rya.indexing.pcj.fluo.app.query.FluoQueryMetadataCache;
    +import org.apache.rya.indexing.pcj.fluo.app.query.MetadataCacheSupplier;
     
     /**
      * This class provides common functionality for implementations of {@link 
BatchBindingSetUpdater}.
      *
      */
     public abstract class AbstractBatchBindingSetUpdater implements 
BatchBindingSetUpdater {
     
    +    protected static final FluoQueryMetadataCache CACHE = 
MetadataCacheSupplier.getOrCreateCache();
    --- End diff --
    
    Add some documentation to this object so that people who extend this class 
know when it is appropriate to use it.


---

Reply via email to