The deleted line was in a method that is invoked in 3 different test:
Thread.Sleep(12000);
And the line converted from constant to injectable property was:
Thread.Sleep(30000);
Since our average build time over the last week is around 11:36, and these two edits save 66 seconds per build, I've achieved a time savings of 9.47% virtually instantly.
So the moral of the story is this: any time you feel a desire to include Thread.Sleep in production code, you need to go stick your head in a freezer until the feeling subsides.
P.S. I think it may be important to note that at least one of those two offending lines was written by me.
No comments:
Post a Comment