[
https://issues.apache.org/jira/browse/HTTPCORE-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16569965#comment-16569965
]
ASF GitHub Bot commented on HTTPCORE-536:
-----------------------------------------
Github user ok2c commented on a diff in the pull request:
https://github.com/apache/httpcomponents-core/pull/72#discussion_r207833571
--- Diff:
httpcore5/src/main/java/org/apache/hc/core5/reactor/DefaultConnectingIOReactor.java
---
@@ -119,28 +115,13 @@ public IOReactorStatus getStatus() {
}
@Override
- public List<ExceptionEvent> getExceptionLog() {
- return auditLog.isEmpty() ?
Collections.<ExceptionEvent>emptyList() : new ArrayList<>(auditLog);
+ public IOWorkers.Selector getWorkerSelector() {
--- End diff --
@clevertension Why do we need to expose `IOWorkers.Selector`? Looks
completely unnecessary to me.
> duplicate connect method could be extract
> -----------------------------------------
>
> Key: HTTPCORE-536
> URL: https://issues.apache.org/jira/browse/HTTPCORE-536
> Project: HttpComponents HttpCore
> Issue Type: Improvement
> Components: HttpCore NIO
> Reporter: silver9886
> Priority: Major
>
> DefaultConnectingIOReactor and DefaultListeningIOReactor connect method are
> same.
> create abstractDefaultIOReactor class,put the connect code to the abstract
> class.
> AsyncServer and AsyncRequester connect method are same.
> create abstractAsyncServerandRequester class,put the connect code to the
> abstract class.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
