Great post! Thanks!

I have also made some comments in the commit.

– Ufuk

> On 09 Dec 2015, at 14:19, Maximilian Michels <m...@apache.org> wrote:
> 
> Hi Matthias,
> 
> Thank you for the blog post. You had already shared a first draft with
> me. This one looks even better!
> 
> I've made some minor comments. +1 to merge if these are addressed.
> 
> Cheers,
> Max
> 
> On Wed, Dec 9, 2015 at 1:20 PM, Matthias J. Sax <mj...@apache.org> wrote:
>> Just updated the draft (thanks to Till and Slim for feedback) and opened
>> a PR.
>> 
>> https://github.com/apache/flink-web/pull/15
>> 
>> @Slim: we discussed about benchmark result beforehand and decided to do
>> a second blog post later on
>> 
>> 
>> -Matthias
>> 
>> On 12/09/2015 12:14 PM, Slim Baltagi wrote:
>>> Matthias,
>>> 
>>> This is great blog!
>>> 
>>> I would like to suggest the following:
>>> Change the title to: How to run your existing Storm applications on Apache 
>>> Flink stream processing engine?
>>> Fixing the few typos
>>> For this reasons -> For these reasons
>>> Storm compatibility package which allows users -> Storm compatibility 
>>> package that allows users
>>> we need to translated it  -> we need to translate it
>>> in not available -> is not available
>>> eg, StormWordCount.jar  -> e.g., StormWordCount.jar
>>> Provide some benchmarks on running a storm application as it is versus 
>>> running it on Flink.
>>> Thanks
>>> 
>>> Slim Baltagi
>>> 
>>> On Dec 9, 2015, at 5:10 AM, Robert Metzger <rmetz...@apache.org> wrote:
>>> 
>>>> Great, thank you for writing the article.
>>>> 
>>>> I like the general idea, but I've found some small typos.
>>>> Can you open a pull request against the "flink-web" repo to make reviewing
>>>> it easier?
>>>> 
>>>> On Wed, Dec 9, 2015 at 11:32 AM, Matthias J. Sax <mj...@apache.org> wrote:
>>>> 
>>>>> Hi,
>>>>> 
>>>>> after talking to several people and getting some feedback already, I
>>>>> would like to suggest a new blog post for the project web site about the
>>>>> Storm compatibility layer.
>>>>> 
>>>>> You can find the draft here:
>>>>> 
>>>>> https://github.com/mjsax/flink-web/blob/stormCompatibilityBlogPost/_posts/2015-12-07-storm-compatibility.md
>>>>> 
>>>>> The missing (just not rendered) picture is this one:
>>>>> 
>>>>> https://github.com/mjsax/flink-web/blob/stormCompatibilityBlogPost/img/blog/flink-storm.png
>>>>> 
>>>>> Looking forward to your feedback!
>>>>> 
>>>>> 
>>>>> -Matthias
>>>>> 
>>>>> 
>>> 
>>> 
>> 

Reply via email to