On 04/24/2013 04:43 PM, Leif Hedstrom wrote:
> On 4/24/13 6:51 AM, Nick Kew wrote:
>> On Fri, 19 Apr 2013 22:54:47 +0100
>> Nick Kew <n...@apache.org> wrote:
>>
>>
>>> I'll attach a mini demo-plugin that reproduces the problem.
>>> The essence of it can be boiled down to:
>> Magic sauce here.
> 
> Indeed. I hate magic.
> 
> So, since for sure the regressions are hanging now, should we back out
> the commit, until we can figure out one of two things:
> 
> 1) If the patch is correct, why are the regressions hanging? And how do
> we fix that?
> 
> 2) If the patch is incorrect, lets figure out what the right solution.
> 
> 
> Thoughts?
> 
> -- Leif
> 
I see ATS freezing up on regression tests in two different places after
this commit was introduced, and, as much as I'm for fixing the original
problem, it's slightly disruptive, so I'm for rolling this back till we
can figure out why it's failing the tests or why it's wrong, whichever
it is. I've tried whatever I can to debug the issue, but apparently, I'm
no good at that :( Waiting for input/output from someone more clever on
this issue :)

With regards,
Daniel.

Reply via email to