[ https://issues.apache.org/jira/browse/DRILL-4841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15553346#comment-15553346 ]
ASF GitHub Bot commented on DRILL-4841: --------------------------------------- Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/565#discussion_r82266114 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillClient.java --- @@ -688,4 +802,142 @@ public DrillBuf getBuffer() { return null; } } + + /** + * Return a new {@link DrillClient.Builder Drill client builder}. + * @return a new builder + */ + public static Builder newBuilder() { + return new Builder(); + } + + /** + * Helper class to construct a {@link DrillClient Drill client}. + */ + public static class Builder { + + private DrillConfig config; + private BufferAllocator allocator; + private ClusterCoordinator clusterCoordinator; + private EventLoopGroup eventLoopGroup; + private ExecutorService executor; + + // defaults + private boolean supportComplexTypes = true; + private boolean isDirectConnection = false; + + /** + * Sets the {@link DrillConfig configuration} for this client. --- End diff -- Scenario? What is used by default? An empty config? One found on the class path? What if I want to use the class-path (default) Drill config, but add my own customizations (as for tests)? Should we explain how to do this, or provide a method to help: withExtraConfig( Config props ) > Use user server event loop group for web clients > ------------------------------------------------ > > Key: DRILL-4841 > URL: https://issues.apache.org/jira/browse/DRILL-4841 > Project: Apache Drill > Issue Type: Improvement > Components: Client - HTTP > Reporter: Sudheesh Katkam > Assignee: Sorabh Hamirwasia > Priority: Minor > > Currently we spawn an event loop group for handling requests from clients. > This group should also be used to handles responses (from server) for web > clients. -- This message was sent by Atlassian JIRA (v6.3.4#6332)