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

    https://github.com/apache/accumulo/pull/224#discussion_r103723706
  
    --- Diff: 
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java 
---
    @@ -808,4 +812,64 @@ void setSamplerConfiguration(String tableName, 
SamplerConfiguration samplerConfi
        * @since 1.8.0
        */
       SamplerConfiguration getSamplerConfiguration(String tableName) throws 
TableNotFoundException, AccumuloException, AccumuloSecurityException;
    +
    +  /**
    +   * This is a entry point for retrieving summaries with optional 
restrictions.
    +   *
    +   * <p>
    +   * Inorder to retrieve Summaries, the Accumulo user making the request 
will need the {@link TablePermission#GET_SUMMARIES} table permission.
    +   *
    +   * <p>
    +   * Accumulo stores summary data with each file in each tablet. In order 
to make retrieving it faster there is a per tablet server cache of summary 
data. The
    +   * size of this cache is determined by the property {code 
tserver.cache.summary.size}. When summary data for a file is not present, it 
will be retrieved using
    +   * threads on the tserver. The property {@code 
tserver.summary.retrieval.threads} determines the max number of threads the 
tserver will use for this.
    +   *
    +   * <p>
    +   * Since summary data is cached, its important to use the summary 
selection options to only read the needed data into the cache.
    +   *
    +   * <p>
    +   * Summary data will be merged on the tablet servers and then in this 
client process. Therefore its important that the required summarizers are on 
the clients
    --- End diff --
    
    s/its/it's/


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to