[ 
https://issues.apache.org/jira/browse/FLINK-13317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16899645#comment-16899645
 ] 

Charles Xu edited comment on FLINK-13317 at 8/4/19 3:02 PM:
------------------------------------------------------------

Should I merge the commit to release-1.9 or apache:master ?  Sorry for that if 
the target version/base is wrong.


was (Author: xuchao):
Should I merge the commit to release-1.9 or apache:master ?  Sorry for that if 
that target version/base is wrong.

> Merge NetUtils and ClientUtils
> ------------------------------
>
>                 Key: FLINK-13317
>                 URL: https://issues.apache.org/jira/browse/FLINK-13317
>             Project: Flink
>          Issue Type: Improvement
>          Components: Command Line Client
>    Affects Versions: 1.8.0, 1.8.1
>            Reporter: Charles Xu
>            Assignee: Charles Xu
>            Priority: Trivial
>              Labels: pull-request-available
>             Fix For: 1.9.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Both NetUtils (flink-core) and ClientUtils (flink-clients) support validating 
> the "host:port" string. To reduce the duplicate code, it's better to move 
> ClientUtils.parseHostPortAddress() to NetUtils class, modify its references 
> and then drop ClientUtils.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to