[ 
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)

Reply via email to