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

    https://github.com/apache/accumulo/pull/159#discussion_r108479204
  
    --- Diff: 
core/src/main/java/org/apache/accumulo/core/iterators/SortedKeyValueIterator.java
 ---
    @@ -147,4 +147,21 @@
        *              if not supported.
        */
       SortedKeyValueIterator<K,V> deepCopy(IteratorEnvironment env);
    +
    +  /**
    +   * Closes any resources that were opened by the 
<tt>SortedKeyValueIterator</tt>. This method does nothing by default. The 
implementing class must close its
    +   * <tt>SortedKeyValueIterator source</tt>. This will be provided in the 
<tt>init</tt> method or by the constructor.
    +   *
    +   * @exception UncheckedIOException
    --- End diff --
    
    After revisiting these changes, I am now being reminded as to why I 
abandoned them months ago...  @keith-turner may have an alternate solution that 
is far less widespread and more manageable. See 
https://issues.apache.org/jira/browse/ACCUMULO-1280


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to