DRILL-1505: Updating Session Value Doesn't Reflect In 'sys.options' Query Output
Project: http://git-wip-us.apache.org/repos/asf/incubator-drill/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-drill/commit/e93fb456 Tree: http://git-wip-us.apache.org/repos/asf/incubator-drill/tree/e93fb456 Diff: http://git-wip-us.apache.org/repos/asf/incubator-drill/diff/e93fb456 Branch: refs/heads/master Commit: e93fb456a4ed096836dd2a60f98c281c44de34c3 Parents: c6ad592 Author: Aditya Kishore <adi...@maprtech.com> Authored: Wed Oct 8 11:23:57 2014 -0700 Committer: Aditya Kishore <adi...@maprtech.com> Committed: Thu Oct 9 13:59:13 2014 -0700 ---------------------------------------------------------------------- .../java/org/apache/drill/exec/server/options/OptionValue.java | 5 +++++ .../apache/drill/exec/server/options/QueryOptionManager.java | 5 ++--- 2 files changed, 7 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-drill/blob/e93fb456/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionValue.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionValue.java b/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionValue.java index 13894ef..bfcbeca 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionValue.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionValue.java @@ -172,4 +172,9 @@ public class OptionValue{ return true; } + @Override + public String toString() { + return "OptionValue [type=" + type + ", name=" + name + ", value=" + getValue() + "]"; + } + } http://git-wip-us.apache.org/repos/asf/incubator-drill/blob/e93fb456/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/QueryOptionManager.java ---------------------------------------------------------------------- diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/QueryOptionManager.java b/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/QueryOptionManager.java index 7eda97e..82fc5ba 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/QueryOptionManager.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/server/options/QueryOptionManager.java @@ -85,9 +85,8 @@ public class QueryOptionManager implements OptionManager { @Override public OptionList getOptionList() { OptionList list = new OptionList(); - for (OptionValue o : options.values()) { - list.add(o); - } + list.addAll(sessionOptions.getOptionList()); + list.addAll(options.values()); return list; }
