Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

minor javadoc fix in JobManager #1358

Closed

Conversation

jaoki
Copy link
Contributor

@jaoki jaoki commented Nov 16, 2015

JobExecutionVertices should be ExecutionJobVertices?
I found it while I have been drawing a Flink local execution diagram.
It is still a half way but let me know if you have any feedback.
https://docs.google.com/drawings/d/1lZg8LkhAlI5lMc2EGCPlYArBmv7cCj-By-phuG6rGE8/edit

@jaoki jaoki changed the title minor jvadoc fix in JobManager minor javadoc fix in JobManager Nov 16, 2015
JobExecutionVertices should be ExecutionJobVertices?
I found it while I have been drawing a Flink local execution diagram.
It is still a half way but let me know if you have any feedback.
https://docs.google.com/drawings/d/1lZg8LkhAlI5lMc2EGCPlYArBmv7cCj-By-phuG6rGE8/edit
@jaoki jaoki force-pushed the minor-javadoc-correction-JobManager branch from 85df104 to 19a15c7 Compare November 16, 2015 08:59
@mxm
Copy link
Contributor

mxm commented Nov 16, 2015

Yes, that's correct. Nice diagram :) Could you post it to the mailing list when you're finished? Thank you!

@jaoki
Copy link
Contributor Author

jaoki commented Nov 16, 2015

@mxm for sure will send to the community once it is done.

@StephanEwen
Copy link
Contributor

Will merge this...

StephanEwen pushed a commit to StephanEwen/flink that referenced this pull request Nov 17, 2015
This closes apache#1358

JobExecutionVertices should be ExecutionJobVertices?
I found it while I have been drawing a Flink local execution diagram.
It is still a half way but let me know if you have any feedback.
https://docs.google.com/drawings/d/1lZg8LkhAlI5lMc2EGCPlYArBmv7cCj-By-phuG6rGE8/edit
@asfgit asfgit closed this in 1f52899 Nov 17, 2015
@jaoki jaoki deleted the minor-javadoc-correction-JobManager branch November 18, 2015 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants