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

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

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

    https://github.com/apache/drill/pull/159#discussion_r40022217
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SessionOptionManager.java
 ---
    @@ -20,14 +20,21 @@
     import com.google.common.base.Predicate;
     import com.google.common.collect.Collections2;
     import org.apache.commons.lang3.tuple.ImmutablePair;
    +import org.apache.drill.common.exceptions.UserException;
     import org.apache.drill.common.map.CaseInsensitiveMap;
     import org.apache.drill.exec.rpc.user.UserSession;
    +import org.apache.drill.exec.server.options.OptionValue.OptionType;
     
     import java.util.Collection;
     import java.util.Map;
     
     /**
    - * {@link OptionManager} that holds options within {@link 
org.apache.drill.exec.rpc.user.UserSession} context.
    + * {@link OptionManager} that holds options within {@link 
org.apache.drill.exec.rpc.user.UserSession} context. Options
    + * set at the session level only apply to queries that you run during the 
current Drill connection. Session level
    + * settings override system level settings.
    + *
    + * NOTE that currently, the effects of deleting a short lived option (see 
{@link OptionValidator#isShortLived}) are
    + * undefined.
    --- End diff --
    
    Should I add a TODO and create a ticket?
    (personally I don't think this is worth a JIRA yet)


> Provide a reset command to reset an option to its default value
> ---------------------------------------------------------------
>
>                 Key: DRILL-1065
>                 URL: https://issues.apache.org/jira/browse/DRILL-1065
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Execution - Flow
>            Reporter: Aman Sinha
>            Assignee: Sudheesh Katkam
>            Priority: Minor
>             Fix For: 1.2.0
>
>
> Within a session, currently we set configuration options and it would be very 
> useful to have a 'reset' command to reset the value of an option to its 
> default system value: 
>   ALTER SESSION RESET <option name> 
> If we don't want to add a new keyword for RESET, we could potentially 
> overload the SET command and allow the user to set to the 'default' value.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to