Thanks for all your feedback! I updated the PR.

I would like to publish the post today. Please let me know if you have
any more comments on the draft.

-Matthias

On 12/09/2015 08:12 PM, Vasiliki Kalavri wrote:
> Thanks Matthias! This is a very nice blog post and reads easily.
> 
> On 9 December 2015 at 19:21, Ufuk Celebi <u...@apache.org> wrote:
> 
>> 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
>>>>>>>
>>>>>>>
>>>>>
>>>>>
>>>>
>>
>>
> 

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to