Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Hammond
Hi,

Upgrade Review Board. The limit is configurable and disabled by default.

Christian



On Jun 21, 2012, at 20:51, Christian Wuerdig  
wrote:

> Hi Christian,
> 
> we just hit that issue here by some reviews missing the diff files.
> 
> In general this is caused when someone works on a feature branch in the 
> repository and then merges in the changes from the trunk. Although it is 
> supposed to happen regularly sometimes there can be two or three weeks 
> between a trunk merge which can amount to larger diffs.
> 
> I would appreciate if that limit can be made configurable
> 
> Cheers
> Christian
> 
> On Thursday, May 17, 2012 7:23:28 AM UTC+12, Christian Hammond wrote:
> Hi Ashish,
> 
> What's the size you're hitting with your diffs, and what's in them? Diffs 
> that are megabytes in size are generally impossible to review.
> 
> I'll look at making the limit flexible, but I want to better understand the 
> problems people are hitting.
> 
> Christian
> 
> -- 
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
> 
> 
> On Wed, May 16, 2012 at 5:14 AM, Ashish Sharma  
> wrote:
> Hi,
> 
> With the release of Review Board (1.6.6), the diff file size is limited to 
> 1MB only.
> 
> Since my setup has a limit of 15MB with earlier version and with the latest 
> update I am facing problems
> 
> How can it be increased to say atleast to 10MB.
> 
> thanks
> Ashish
> -- 
> Want to help the Review Board project? Donate today at 
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en
> 
> -- 
> Want to help the Review Board project? Donate today at 
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Wuerdig
Sorry, ignore that, just realized that it is configurable since 1.6.7

On Friday, June 22, 2012 3:51:56 PM UTC+12, Christian Wuerdig wrote:
>
> Hi Christian,
>
> we just hit that issue here by some reviews missing the diff files.
>
> In general this is caused when someone works on a feature branch in the 
> repository and then merges in the changes from the trunk. Although it is 
> supposed to happen regularly sometimes there can be two or three weeks 
> between a trunk merge which can amount to larger diffs.
>
> I would appreciate if that limit can be made configurable
>
> Cheers
> Christian
>
> On Thursday, May 17, 2012 7:23:28 AM UTC+12, Christian Hammond wrote:
>>
>> Hi Ashish,
>>
>> What's the size you're hitting with your diffs, and what's in them? Diffs 
>> that are megabytes in size are generally impossible to review.
>>
>> I'll look at making the limit flexible, but I want to better understand 
>> the problems people are hitting.
>>
>> Christian
>>
>> -- 
>> Christian Hammond - chip...@chipx86.com
>> Review Board - http://www.reviewboard.org
>> VMware, Inc. - http://www.vmware.com
>>
>>
>> On Wed, May 16, 2012 at 5:14 AM, Ashish Sharma 
>> wrote:
>>
>>> Hi,
>>>
>>> With the release of Review Board (1.6.6), the diff file size is limited 
>>> to 1MB only.
>>>
>>> Since my setup has a limit of 15MB with earlier version and with the 
>>> latest update I am facing problems
>>>
>>> How can it be increased to say atleast to 10MB.
>>>
>>> thanks
>>> Ashish
>>>
>>> -- 
>>> Want to help the Review Board project? Donate today at 
>>> http://www.reviewboard.org/donate/
>>> Happy user? Let us know at http://www.reviewboard.org/users/
>>> -~--~~~~--~~--~--~---
>>> To unsubscribe from this group, send email to 
>>> reviewboard+unsubscr...@googlegroups.com
>>> For more options, visit this group at 
>>> http://groups.google.com/group/reviewboard?hl=en
>>
>>
>>

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Diff fie size limited to 1 MB

2012-06-21 Thread Christian Wuerdig
Hi Christian,

we just hit that issue here by some reviews missing the diff files.

In general this is caused when someone works on a feature branch in the 
repository and then merges in the changes from the trunk. Although it is 
supposed to happen regularly sometimes there can be two or three weeks 
between a trunk merge which can amount to larger diffs.

I would appreciate if that limit can be made configurable

Cheers
Christian

On Thursday, May 17, 2012 7:23:28 AM UTC+12, Christian Hammond wrote:
>
> Hi Ashish,
>
> What's the size you're hitting with your diffs, and what's in them? Diffs 
> that are megabytes in size are generally impossible to review.
>
> I'll look at making the limit flexible, but I want to better understand 
> the problems people are hitting.
>
> Christian
>
> -- 
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Wed, May 16, 2012 at 5:14 AM, Ashish Sharma 
> wrote:
>
>> Hi,
>>
>> With the release of Review Board (1.6.6), the diff file size is limited 
>> to 1MB only.
>>
>> Since my setup has a limit of 15MB with earlier version and with the 
>> latest update I am facing problems
>>
>> How can it be increased to say atleast to 10MB.
>>
>> thanks
>> Ashish
>>
>> -- 
>> Want to help the Review Board project? Donate today at 
>> http://www.reviewboard.org/donate/
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> -~--~~~~--~~--~--~---
>> To unsubscribe from this group, send email to 
>> reviewboard+unsubscr...@googlegroups.com
>> For more options, visit this group at 
>> http://groups.google.com/group/reviewboard?hl=en
>
>
>

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Diff fie size limited to 1 MB

2012-05-19 Thread Ashish Sharma
Christian,

You are right in your observation that Diff's with large size are
impossible to review, following are some of the reasons that can help you
understand for the need of large diff sizes (as per my own experience):

1. There may be sample test files in the diff itself that usually increase
the size.
2. Since the earlier versions didn't have much of a size limit, so the
users just didn't care about the size limitation and during the upgrade,
they are hit hard with the diff size limitation, change should be gradual.
3. Simple solution here is to divide large file checkin from the reviewable
code but it's still difficult to communicate it over a large team size at
once and enforce on a small time scale.

thanks
Ashish

On Thu, May 17, 2012 at 12:53 AM, Christian Hammond wrote:

> Hi Ashish,
>
> What's the size you're hitting with your diffs, and what's in them? Diffs
> that are megabytes in size are generally impossible to review.
>
> I'll look at making the limit flexible, but I want to better understand
> the problems people are hitting.
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Wed, May 16, 2012 at 5:14 AM, Ashish Sharma 
> wrote:
>
>> Hi,
>>
>> With the release of Review Board (1.6.6), the diff file size is limited
>> to 1MB only.
>>
>> Since my setup has a limit of 15MB with earlier version and with the
>> latest update I am facing problems
>>
>> How can it be increased to say atleast to 10MB.
>>
>> thanks
>>
>> Ashish
>>
>> --
>> Want to help the Review Board project? Donate today at
>> http://www.reviewboard.org/donate/
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> -~--~~~~--~~--~--~---
>> To unsubscribe from this group, send email to
>> reviewboard+unsubscr...@googlegroups.com
>> For more options, visit this group at
>> http://groups.google.com/group/reviewboard?hl=en
>
>
>  --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Diff fie size limited to 1 MB

2012-05-16 Thread Christian Hammond
Hi Ashish,

What's the size you're hitting with your diffs, and what's in them? Diffs
that are megabytes in size are generally impossible to review.

I'll look at making the limit flexible, but I want to better understand the
problems people are hitting.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Wed, May 16, 2012 at 5:14 AM, Ashish Sharma wrote:

> Hi,
>
> With the release of Review Board (1.6.6), the diff file size is limited to
> 1MB only.
>
> Since my setup has a limit of 15MB with earlier version and with the
> latest update I am facing problems
>
> How can it be increased to say atleast to 10MB.
>
> thanks
> Ashish
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en