Skip to content

Conversation

greghogan
Copy link
Contributor

Provide descriptive operator names for Graph and GraphCsvReader.
Condense multiple type conversion maps into a single mapper.
Reuse objects in operations wrapping user-defined-functions.

Travis CI tests are currently broken due to an SSL certificate error at https://repo.maven.apache.org/maven2

Provide descriptive operator names for Graph and GraphCsvReader.
Condense multiple type conversion maps into a single mapper.
Reuse objects in operations wrapping user-defined-functions.
@greghogan
Copy link
Contributor Author

@vasia, would you like to review this?

@vasia
Copy link
Contributor

vasia commented Dec 6, 2016

Yes, will do in the following days, thanks!

Copy link
Contributor

@vasia vasia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me, thanks @greghogan!

@greghogan greghogan mentioned this pull request Dec 12, 2016
3 tasks
@greghogan
Copy link
Contributor Author

That's odd, your last comment was not propagated to JIRA @vasia. Might have been an outage at that time. Will merge this ...

@asfgit asfgit closed this in 09e0817 Dec 14, 2016
joseprupi pushed a commit to joseprupi/flink that referenced this pull request Feb 12, 2017
Provide descriptive operator names for Graph and GraphCsvReader.
Condense multiple type conversion maps into a single mapper.
Reuse objects in operations wrapping user-defined-functions.

This closes apache#2832
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