Skip to content

FLINK-6041[flink-streaming-java] Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'#3532

Closed
liuyuzhong7 wants to merge 2 commits intoapache:masterfrom
liuyuzhong7:FLINK-6041
Closed

FLINK-6041[flink-streaming-java] Move StreamingFunctionUtils to 'org.apache.flink.streaming.util'#3532
liuyuzhong7 wants to merge 2 commits intoapache:masterfrom
liuyuzhong7:FLINK-6041

Conversation

@liuyuzhong7
Copy link
Copy Markdown
Contributor

@liuyuzhong7 liuyuzhong7 commented Mar 14, 2017

StreamingFunctionUtils.java is in 'org.apache.flink.streaming.api.functions.util'
And there also a package 'org.apache.flink.streaming.util' in project.
I think move StreamingFunctionUtils.java to 'org.apache.flink.streaming.api.util.functions' is make code more clear.

@StephanEwen
Copy link
Copy Markdown
Contributor

Agreed, the utilities are not part of the API, so they should not be in a subdirectory of the API.

Merging this...

StephanEwen pushed a commit to StephanEwen/flink that referenced this pull request Mar 16, 2017
@asfgit asfgit closed this in 03889ae Mar 16, 2017
stefanobortoli pushed a commit to huawei-flink/flink that referenced this pull request Mar 20, 2017
p16i pushed a commit to p16i/flink that referenced this pull request Apr 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants