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