drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-4841) Use user server event loop group for web clients
Date Thu, 06 Oct 2016 22:08:21 GMT

    [ https://issues.apache.org/jira/browse/DRILL-4841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15553344#comment-15553344

ASF GitHub Bot commented on DRILL-4841:

Github user paul-rogers commented on a diff in the pull request:

    --- Diff: common/src/main/java/org/apache/drill/common/AutoCloseables.java ---
    @@ -87,4 +87,29 @@ public static void close(Iterable<? extends AutoCloseable> ac)
throws Exception
           throw topLevelException;
    +  /**
    +   * close() an {@see java.lang.AutoCloseable} without throwing a (checked)
    +   * {@see java.lang.Exception}. This wraps the close() call with a
    +   * try-catch that will rethrow an Exception wrapped with a
    +   * {@see java.lang.RuntimeException}, providing a way to call close()
    +   * without having to do the try-catch everywhere or propagate the Exception.
    +   *
    +   * @param autoCloseable the AutoCloseable to close; may be null
    +   * @throws RuntimeException if an Exception occurs; the Exception is
    +   *   wrapped by the RuntimeException
    +   */
    +  public static void closeNoChecked(final AutoCloseable autoCloseable) {
    --- End diff --
    closeUnchecked ?
    But, we are "checking", the "checked" has to do with the Exception type.
    Maybe cleanClose( ) for this function. Then, add a "closeSilently" to catch and ignore
close exceptions. (The closeSilently is handy in the case when, say, a file is full, a write
failed, and the close will also fail because it still can't flush pending buffers.) There
are other functions to the to silent close, but might be handy to have them in one place.

> 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

View raw message