You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Arpit Agarwal (JIRA)" <ji...@apache.org> on 2015/07/25 01:17:06 UTC
[jira] [Commented] (HADOOP-12272) Refactor ipc.Server and
implementations to reduce constructor bloat
[ https://issues.apache.org/jira/browse/HADOOP-12272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14641222#comment-14641222 ]
Arpit Agarwal commented on HADOOP-12272:
----------------------------------------
Ran into this while adding a new Server constructor parameter for HADOOP-12250. Will probably perform the refactoring after fixing HADOOP-12250.
I have an incomplete patch to refactor the ipc code but need to fix usages in the rest of the code.
> Refactor ipc.Server and implementations to reduce constructor bloat
> -------------------------------------------------------------------
>
> Key: HADOOP-12272
> URL: https://issues.apache.org/jira/browse/HADOOP-12272
> Project: Hadoop Common
> Issue Type: Bug
> Components: ipc
> Reporter: Arpit Agarwal
>
> {{ipc.Server}} and its implementations have constructors taking large number of parameters. This code can be simplified quite a bit by just moving RPC.Builder to the Server class and passing the builder object to constructors.
> The refactoring should be safe based on the class annotations but need to confirm no dependent components will break.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)