[ https://issues.apache.org/jira/browse/DRILL-1065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14804513#comment-14804513 ]
ASF GitHub Bot commented on DRILL-1065: --------------------------------------- Github user adeneche commented on a diff in the pull request: https://github.com/apache/drill/pull/159#discussion_r39799140 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java --- @@ -241,6 +243,31 @@ public void setOption(final OptionValue value) { } @Override + public void deleteOption(final String name, OptionType type) { + assert type == OptionType.SYSTEM; + try { // ensure option exists + getValidator(name); + } catch (final IllegalArgumentException e) { + throw UserException.validationError() --- End diff -- you can just use: ``` throw UserException.validationError(e).build(logger); ``` > 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)