Re: timeout is now requested to be on all tests

2013-02-21 Thread Surenkumar Nihalani
Just wanted to give everyone a heads up. Due to this change, I have introduced a bug. It's returning opposite value of what it's supposed to. So, +1 of all tests you change brings up a -1 overall and vice versa. I have reopened HADOOP-9112 and uploaded the patch to reverse to reverse to the

Re: timeout is now requested to be on all tests

2013-02-21 Thread Steve Loughran
thanks, just seen and commented on this. IF we're going to have test timeouts, we need a good recommended default value for all tests except the extra slow ones. Or 1. we just use our own JUnit fork that sets up a better default value than 0. I don't know how Ken would react to that. 2. we get ma

timeout is now requested to be on all tests

2013-02-20 Thread Robert Evans
Sorry about cross posting, but this will impact all developers and I wanted to give you all a heads-up. HADOOP-9112 was just checked it. This means that the pre commit build will now give a –1 for any patch with junit tests that do not include