http://ayende.comAyende @ RahienCopyright (C) Ayende Rahien 2004 - 2021 (c) 202660surely a check in with 1695 changes is too big. I think that another unit of measure to find "big" change set could be the time that pass between the instant you modify a file, and the time you chek in the repository. More is the time you take the file with changes in your local copy, higher the chance of confilcts. I always suggest people to have a branch in situation where they needs more time to reach a stable point. In this way you still deal with rapid checkin and you do not affect the trunk. I prefer small and fast checkin, expecially when you link the check in with an issue in the bug tracker, just correct the bug, and fire a check in. Alk. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment13http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment13Sat, 26 Apr 2008 06:48:12 GMTWhat if you do some sort of global change to the application though that's really smally? For example, maybe I change the root namespace for some reason. It hits every single file in the application but only changes one or two lines per file and it's a predictable change. I think that as long as that were the only change in that check in it would be fine. Still would be annoying, but I don't have a problem with it. On the other hand, if someone checks in 1200 files and there are 50 new features associated with that check in, someone needs to check code in more often. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment12http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment12Thu, 24 Apr 2008 21:16:46 GMTAdam, Yes, you hit the major point about why I dislike distributed SCM http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment11http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment11Thu, 24 Apr 2008 15:03:42 GMTEven with small local commits with git or to a branch, how can a maintainer easily review the final "BIG" commit to the working trunk? I would use git and prefer to review 100 tiny checkins with comments over a single commit of 1600+ changes. So I think git is still the answer. Either way a major refactoring like that is going to be tough to review regardless of the tools or process. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment10http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment10Thu, 24 Apr 2008 14:43:52 GMTIt's been a while, but isn't a merge in svn done as one big commit? IMO the reason for smaller checkins in git are because it is so much faster. The time it takes to do git add . git ci -m "" is trivial, so frequent commits are easy. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment9http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment9Thu, 24 Apr 2008 13:40:50 GMT@Will, That line of thinking is just plain WRONG http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment8http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment8Thu, 24 Apr 2008 12:27:06 GMTForgive me if i'm missing the obvious, but isn't this what branches are for? The 'big chicken' (thats how i read it too :P) developer should have had his own branch to make his incremental changes whilst still checking in and not disrupting the main branch. Unless of course it's as per Jason's suggestion and was a find/replace, but even then source control is not a replacement for team communications. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment7http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment7Thu, 24 Apr 2008 11:29:37 GMTThat's a big check-in. The wide name changes is one explanation, or too lazy to check-in neatly divided features. But another plausible reason that Git solves is the large experiments that the developer is unsure will work in the long term - why ever check that in?! When the experiment eventually works out - what are they to do? Clearly, they just check it in! The same thing happened to me today, but the PM saw it coming. Isn't this an inherent limitation of non-GIT/Distributed source control? http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment6http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment6Thu, 24 Apr 2008 09:04:19 GMTI first read that: "Prefer small CHICKENS, they are easier to deal with." Which is probably true... http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment5http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment5Thu, 24 Apr 2008 07:43:49 GMTIt wasn't me, but sounds like a huge refactoring. But I'll recommend the next time "someone" should checkin a bit earlier. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment4http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment4Thu, 24 Apr 2008 07:17:23 GMTI totally agree that big check-ins are a bad idea but is the file count an accurate measure of a big check-in? Perhaps, purely for example, I did a global find and replace on the namespace prefix thoughout a solution and checked in... this would probably touch every source file, project file, and build script. However, in each file it would only be a small change to a single line. http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment3http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment3Thu, 24 Apr 2008 07:11:25 GMTThat sounds like a post for the daily WTF! http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment2http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment2Thu, 24 Apr 2008 03:41:00 GMTI've moved over to Git now, and am really digging the workflow. I've also noticed now that my commits have become more concise and focused around particular features. Though I can't point my finger on _exactly_ why..... ps. GitBridge? ; ) ~Dave http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment1http://ayende.com/3286/source-control-practices-to-be-abolished-the-large-checkin#comment1Thu, 24 Apr 2008 03:11:30 GMT