Jason Altekruse created DRILL-3521:
--------------------------------------

             Summary: [umbrella] Review switch statements throughout codebase 
to add default cases where there are none
                 Key: DRILL-3521
                 URL: https://issues.apache.org/jira/browse/DRILL-3521
             Project: Apache Drill
          Issue Type: Bug
          Components: Execution - Flow
            Reporter: Jason Altekruse
            Assignee: Jason Altekruse


There are a number of places in the code that are missing default branches on 
case statements. One particular instance I noticed is in 
OptionValue.createOption, which returns null if passed an unexpected type. This 
and a few other places in the code could be made a little nicer to work with if 
we just provided the standard behavior of throwing an exception.

One additional note, in a number of places where we do have defaults, but the 
exception thrown is an UnsupportedOperationException with no message. 
DRILL-2680 discusses this problem, so this might be handled over there, but as 
we fill in the switch defaults we should try to avoid introducing the new 
problem of exceptions lacking descriptions.



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

Reply via email to