I think this is too much stuff for a ternary assignment,
```
Response.Status status;
String errMsg;
if (spec.isPresent()) {
...
```
would be easier to follow and wouldn't double check `spec.isPresent`

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6272 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to