[DRILL-7673] View set query fails with NPE for non-existing option

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.17.0
• Fix Version/s: 1.18.0
• Labels: ready-to-commit
• Assignee: Vova Vysotskyi
• Reporter: Vova Vysotskyi
• Reviewer: Paul Rogers
• Created: 27/Mar/20 23:05
• Updated: 30/Mar/20 20:45
• Resolved: 30/Mar/20 20:45

Description

The following query fails with NPE:

set `non-existing option`

Stack trace:

Caused by: java.lang.NullPointerException: null
	at org.apache.drill.exec.planner.sql.handlers.SetOptionHandler.getPlan(SetOptionHandler.java:66)
	at org.apache.drill.exec.planner.sql.DrillSqlWorker.getQueryPlan(DrillSqlWorker.java:283)
	at org.apache.drill.exec.planner.sql.DrillSqlWorker.getPhysicalPlan(DrillSqlWorker.java:163)
	at org.apache.drill.exec.planner.sql.DrillSqlWorker.convertPlan(DrillSqlWorker.java:140)
	at org.apache.drill.exec.planner.sql.DrillSqlWorker.getPlan(DrillSqlWorker.java:93)
	at org.apache.drill.exec.work.foreman.Foreman.runSQL(Foreman.java:592)
	at org.apache.drill.exec.work.foreman.Foreman.run(Foreman.java:277)
	... 3 common frames omitted

Also, the result of the command returns option name with the same case as it was specified in the query:

set `metastore.ENABLED`;
+-------------------+-------+
|       name        | value |
+-------------------+-------+
| metastore.ENABLED | false |
+-------------------+-------+

Issue Links

• links to: GitHub Pull Request #2043