[ 
https://issues.apache.org/jira/browse/DRILL-5481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16007313#comment-16007313
 ] 

ASF GitHub Bot commented on DRILL-5481:
---------------------------------------

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

    https://github.com/apache/drill/pull/827#discussion_r116116596
  
    --- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
    @@ -138,7 +138,10 @@ drill.exec: {
         class: 
"org.apache.drill.exec.store.sys.store.provider.ZookeeperPersistentStoreProvider",
         local: {
           path: "/tmp/drill",
    -      write: true
    +      inmemory: {
    +        write: false,
    +        capacity: 1000
    --- End diff --
    
    Remove this line, otherwise `if 
(config.hasPath(ExecConstants.SYS_STORE_CAPACITY_PROFILES))` is always true.


> Allow Drill to persist profiles in-memory only with a max capacity
> ------------------------------------------------------------------
>
>                 Key: DRILL-5481
>                 URL: https://issues.apache.org/jira/browse/DRILL-5481
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.10.0
>            Reporter: Kunal Khatua
>            Assignee: Kunal Khatua
>
> To allow for fast persistence of profiles on a temporary basis (i.e. till the 
> life of the Drillbit), an existing test class 
> {{org.apache.drill.exec.testing.store.NoWriteLocalStore.java}} was refactored 
> to {{org.apache.drill.exec.store.sys.store.EphemeralPersistentStore}} and 
> given the ability to maintain a max capacity.
> This should allow query profiles to be available for as long as the Drillbit 
> process' lifespan.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to