Hi Faraaz,

Thanks for your patience. I am looking at patches if I can spend a little time.

While your patches are of a very good quality, two packages blew up when I 
installed them.
Please fix these issues and I will apply the packaging to bigtop. Further 
testing is the job of the smoke tests.

Olaf


> Am 19.02.2016 um 22:51 schrieb Faraaz Sareshwala <[email protected]>:
> 
> I’ve made all the requested changes for the submitted patches. I think the 
> puppet recipes and smoke tests are pretty much ready to go. Let me know if 
> you guys have any comments on that. Can someone take a look at the packaging 
> so we can get that reviewed and merged?
> 
> Thanks!
> 
> 
> 
> On 2/17/16, 11:49 AM, "Faraaz Sareshwala" <[email protected]> wrote:
> 
>> No worries :). I’ve gotten a couple of review comments from Olaf and Cos. 
>> They’re keeping me busy enough.
>> 
>> Would love for you to take a look when you can though!
>> 
>> Faraaz
>> 
>> 
>> 
>> 
>> On 2/17/16, 10:04 AM, "Evans Ye" <[email protected]> wrote:
>> 
>>> Hi Faraaz,
>>> 
>>> Thanks for delivering patches this quick.
>>> As you can see most of our people is mainly working on the release, hence
>>> we mind not be able to take care everything in a timely manner during this
>>> period.
>>> Anyhow, either I or other community member will review your patches and get
>>> it merged.
>>> But I myself still have some pending JIRAs in my queue waiting for review
>>> and merge.
>>> So please hang on. :)
>>> 
>>> 
>>> 
>>> 2016-02-08 10:22 GMT+08:00 Faraaz Sareshwala <[email protected]>:
>>> 
>>>> Sorry for the second email. Just want to mention that I have one last
>>>> patch coming up for qfs smoke tests. I haven’t finished that just yet. But
>>>> I wanted to get things moving for the two patches that are already ready.
>>>> 
>>>> 
>>>> 
>>>> 
>>>> On 2/7/16, 6:18 PM, "Faraaz Sareshwala" <[email protected]> wrote:
>>>> 
>>>>> Hi guys,
>>>>> 
>>>>> I’ve got two patches that I’d like to get reviewed and committed into the
>>>> BigTop repository. These patches contain code to include qfs (Quantcast
>>>> FileSystem) into the BigTop ecosystem. I’ve written the code to package qfs
>>>> for both rpm and debian based systems. I’ve also written the puppet recipes
>>>> to make sure that components are able to talk to each other correctly.
>>>>> 
>>>>> If you have some time, could you please review these patches and offer
>>>> any feedback you have so that we can get these merged in?
>>>>> 
>>>>> Thank you so much!
>>>>> 
>>>>> Faraaz
>>>>> 
>>>>> https://issues.apache.org/jira/browse/BIGTOP-2285
>>>>> 
>>>>> https://issues.apache.org/jira/browse/BIGTOP-2293
>>>>> 
>>>>> 
>>>> 

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to