Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/1040#discussion_r156645220
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebServer.java
---
@@ -183,17 +180,19 @@ public void start() throws Exception {
}
}
- private ServletContextHandler createServletContextHandler(final boolean
authEnabled) {
+ private ServletContextHandler createServletContextHandler(final boolean
authEnabled) throws Exception {
--- End diff --
`DrillHttpSecurityHandlerProvider` constructor throws
`DrillbitStartupException`, it seems better to re-throw exact exception rather
then broad `Exception`.
---
