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

[blog] flink on zeppelin - part2 #344

Closed
wants to merge 5 commits into from

Conversation

zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Jun 2, 2020

This is part2 of flink on zeppelin blog.

@zjffdu
Copy link
Contributor Author

zjffdu commented Jun 2, 2020

@morsapaes @MarkSfik This is part2 of flink on zeppelin

### Single Mode
Single mode is for the case when the result of sql statement is always one row, such as the following example.
The output format is HTML, and you can specify paragraph local property template for the final output content template.
And you can use {i} as placeholder for the ith column of result.
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure what "ith" maybe you want to ad {i}th if you want to say 9th or 8th?
It should also be "column of the result"

Copy link
Contributor

@MarkSfik MarkSfik left a comment

Choose a reason for hiding this comment

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

Thanks for sharing the second part of the story @zjffdu
I left some suggestions and comments here.

zjffdu and others added 2 commits June 4, 2020 14:17
Co-authored-by: MarkSfik <47176197+MarkSfik@users.noreply.github.com>
@zjffdu
Copy link
Contributor Author

zjffdu commented Jun 4, 2020

Thanks for the review @MarkSfik , I have updated the post

Copy link
Contributor

@morsapaes morsapaes left a comment

Choose a reason for hiding this comment

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

Thanks for this follow-up blogpost, @zjffdu! I think we can leave some time between publishing the blogposts instead of publishing both at the same time, what do you think?

_posts/2020-05-25-flink-on-zeppelin-part2.md Outdated Show resolved Hide resolved
_posts/2020-05-25-flink-on-zeppelin-part2.md Outdated Show resolved Hide resolved
_posts/2020-05-25-flink-on-zeppelin-part2.md Outdated Show resolved Hide resolved
_posts/2020-05-25-flink-on-zeppelin-part2.md Outdated Show resolved Hide resolved
Co-authored-by: morsapaes <marta.paes.moreira@gmail.com>
@zjffdu
Copy link
Contributor Author

zjffdu commented Jun 8, 2020

Thanks @morsapaes for the review, I have updated post, publishing separately at different time make sense to me. How about publishing part-1 this week, and part-2 next week ?

@morsapaes
Copy link
Contributor

Sounds good to me, @zjffdu!

Co-authored-by: morsapaes <marta.paes.moreira@gmail.com>
@zjffdu
Copy link
Contributor Author

zjffdu commented Jun 19, 2020

Thanks @morsapaes

Copy link
Member

@carp84 carp84 left a comment

Choose a reason for hiding this comment

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

+1, thanks for the contribution @zjffdu and @morsapaes !

Will merge this with some minor adjustments to coordinate the file name of the blog markdown with the date in blog header, as well as image directory name, to make sure the blog content appears in the right place on our website.

@carp84 carp84 closed this in a5fea70 Jun 25, 2020
@zjffdu
Copy link
Contributor Author

zjffdu commented Jun 25, 2020

@carp84 Thanks for helping merging this.

XComp pushed a commit to XComp/flink-web that referenced this pull request Sep 7, 2020
Co-authored-by: morsapaes <marta.paes.moreira@gmail.com>

This closes apache#344.
curcur pushed a commit to curcur/flink-web that referenced this pull request Apr 5, 2021
Co-authored-by: morsapaes <marta.paes.moreira@gmail.com>

This closes apache#344.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants