Github user vrozov commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1148#discussion_r171989096
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/WebServer.java 
---
    @@ -424,10 +418,11 @@ private ServerConnector createHttpsConnector(int 
port) throws Exception {
        * @return Initialized {@link ServerConnector} instance for HTTP 
connections.
        * @throws Exception
        */
    -  private ServerConnector createHttpConnector(int port) throws Exception {
    +  private ServerConnector createHttpConnector(int port, int acceptors, int 
selectors) throws Exception {
         logger.info("Setting up HTTP connector for web server");
         final HttpConfiguration httpConfig = new HttpConfiguration();
    -    final ServerConnector httpConnector = new 
ServerConnector(embeddedJetty, new HttpConnectionFactory(httpConfig));
    +    final ServerConnector httpConnector =
    +        new ServerConnector(embeddedJetty, null, null, null, acceptors, 
selectors, new HttpConnectionFactory(httpConfig));
    --- End diff --
    
    I don't think there is such constructor (only one that takes 
`SslContextFactory` as the last argument), please double check. 


---

Reply via email to