You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Guozhang Wang (JIRA)" <ji...@apache.org> on 2016/02/22 23:16:18 UTC
[jira] [Updated] (KAFKA-3261) Consolidate class
kafka.cluster.BrokerEndPoint and kafka.cluster.EndPoint
[ https://issues.apache.org/jira/browse/KAFKA-3261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Guozhang Wang updated KAFKA-3261:
---------------------------------
Description:
These two classes are serving similar purposes and can be consolidated. Also as [~sasakitoa] suggested we can remove their "uriParseExp" variables but use (a possibly modified)
{code}
private static final Pattern HOST_PORT_PATTERN = Pattern.compile(".*?\\[?([0-9a-zA-Z\\-.:]*)\\]?:([0-9]+)");
{code}
in org.apache.kafka.common.utils.Utils instead.
was:
These two classes are serving similar purposes and can be consolidated. Also we can remove their "uriParseExp" variable but use
{code}
private static final Pattern HOST_PORT_PATTERN = Pattern.compile(".*?\\[?([0-9a-zA-Z\\-.:]*)\\]?:([0-9]+)");
{code}
in org.apache.kafka.common.utils.Utils instead.
> Consolidate class kafka.cluster.BrokerEndPoint and kafka.cluster.EndPoint
> -------------------------------------------------------------------------
>
> Key: KAFKA-3261
> URL: https://issues.apache.org/jira/browse/KAFKA-3261
> Project: Kafka
> Issue Type: Bug
> Reporter: Guozhang Wang
>
> These two classes are serving similar purposes and can be consolidated. Also as [~sasakitoa] suggested we can remove their "uriParseExp" variables but use (a possibly modified)
> {code}
> private static final Pattern HOST_PORT_PATTERN = Pattern.compile(".*?\\[?([0-9a-zA-Z\\-.:]*)\\]?:([0-9]+)");
> {code}
> in org.apache.kafka.common.utils.Utils instead.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)