19

Definitely a bug. One question on SO was migrated with six close votes. It ended up making two questions on meta after migration.

Is this a race condition with the sixth vote causing the question to be migrated a second time? Could be fixed by either making sure six people can't close a question, or by checking to see if a question was already migrated before migrating it. Assuming that's what the problem is (I've been known to be wrong before).

Both questions on meta link back to the same question on SO, so they were definitely migrated twice from the same source. Additionally, the OP's profile contains only one closed question.

Further evidence: the revision history of the original question is pretty clearly showing that it's been migrated twice because of the sixth close vote:

Post Migrated to meta.stackoverflow.com by Konrad Rudolph, Naveen, Shoban, Daniel Brückner, Yuval F

Post Migrated to meta.stackoverflow.com by Konrad Rudolph, Naveen, Shoban, Daniel Brückner, Yuval F, SilentGhost

Emphasis mine.

Looks like it's not fixed afterall: https://stackoverflow.com/revisions/2030983/list

Was migrated twice due to six close votes. I'm very disappointed in you, Angels.

|improve this question
  • ... which answers the question who would be the first to point out this bug (I had mine typed up already -- well, you beat me). – balpha Sep 1 '09 at 11:29
  • Bwahaha! Did you come to the same conclusion that I did? – Welbog Sep 1 '09 at 11:30
  • 2
    Also, shame on you for trying to beat me, Welbog, at a typing race. I'm like a word wizard. A viscount of vocabulary. A sorcerer of salutations! – Welbog Sep 1 '09 at 11:31
  • 5
    Na, I had a whole patch to the SO source code worked out which fixed the issue. It also made the badges a little shinier. But oh well... your question is fine, I guess. – balpha Sep 1 '09 at 11:37
  • I deleted the dup - but it definitely happened; I deliberately didn't "merge" it; even though that would have brought the comments over, it would have lost visibility of the 2nd question's existence. – Marc Gravell♦ Sep 1 '09 at 11:37
  • I appreciate that, Marc. But couldn't you have just closed and locked it instead so that everyone can see it? – Welbog Sep 1 '09 at 11:43
  • 2
    @balpha: Don't be so slow next time. I wrote Jeff Atwood 38 years ago in order to create something that would create badges in the first place. – Welbog Sep 1 '09 at 11:45
  • Fine; closed and locked... not sure what people will see that is very different, though. – Marc Gravell♦ Sep 1 '09 at 12:28
  • 1
    Well, if it's visible it's more likely people will understand there's actually a problem and might have more insight into it. I don't know. Do what you want. I'm not your mother. – Welbog Sep 1 '09 at 12:38
  • 2
    No mother's day gift for you, then. Besides, there's no real harm in having it dup for a while... (since it can't do anything while closed+locked) – Marc Gravell♦ Sep 1 '09 at 12:40
  • That's the spirit! – Welbog Sep 1 '09 at 12:52
  • 2
    I feel like "since it can't do anything while closed and locked" is the opening for a cheesy horror movie, MSO style. – Eric Sep 1 '09 at 13:00
  • 1
    @Eric: We should make sure that there are no Zombie Questions... Get your close-shotgun handy! – perbert Sep 1 '09 at 13:12
  • Again, looks like this one isn't fixed: meta.stackexchange.com/questions/47244/… meta.stackexchange.com/questions/47243/… – Welbog Apr 20 '10 at 14:24
  • It's showed up again, but in fiercer force than ever before! stackoverflow.com/questions/2915563/… has been migrated to meta 6 times now! This post is the one with answers in it. – Grace Note♦ May 26 '10 at 18:20
7

Yes, this is a bug - for close votes, we're going to have to "lock" a question during the vote. This won't affect other vote types.

Good find on this - working on it now!

Update - when fixing this, I'm shocked to admit I used a goto statement for the first time ever - I'd post the code, but I'm kinda ashamed :-P

But this problem is hopefully fixed on the next deployment!

|improve this answer
  • 1
    Good to know! Thanks for the heads up. – Welbog Sep 4 '09 at 1:21
  • 12
    Post it! Post it! Post it! – Super Long Names are Hilarious Sep 4 '09 at 3:19
  • 3
    Alas, programmers everywhere would see the code and then flee our site for another - probably one allowing professional gender reassignment. – Jarrod Dixon♦ Sep 4 '09 at 5:31
  • Also, I could have used a while (true) loop, but I find that just as hacky as goto - this case was extremely localized; only 7 lines of code for the fix. – Jarrod Dixon♦ Sep 4 '09 at 5:34
  • 1
    xkcd.com/292/- At my last job, doing COBOL, this got taped up to the wall. – AgentConundrum Sep 4 '09 at 5:47
  • @AgentConundrum - thank goodness I've got anti-raptor spray! – Jarrod Dixon♦ Sep 4 '09 at 6:28
  • 1
    I can't upvote this. Not with a goto in it! – Ladybug Killer Sep 4 '09 at 7:29
  • +1 I'll up-vote only because it's Jarrod. You don't get much opportunity to too otherwise. But GOTO? I suddenly see stream of the SO code-base in VB.Net. – BinaryMisfit Sep 4 '09 at 10:26
  • 7
    POST THE CODE, MAN – Welbog Sep 4 '09 at 12:50
  • 1
    Hey, this issue is back again: meta.stackexchange.com/questions/47243/… meta.stackexchange.com/questions/47244/… – Welbog Apr 20 '10 at 14:24
  • 4
    The goto has been removed - I can sleep again. – Jarrod Dixon♦ Jun 1 '10 at 5:18
0

Couldn't it have something to do with the editing of the question? even though they are some ms apart, that's what could have triggered this.

|improve this answer
  • I doubt it. Take a look at the original question's revisions: stackoverflow.com/revisions/1361881/list It's pretty clear that it was migrated once because Konrad Rudolph, Naveen, Shoban, Daniel Brückner, Yuval F voted to migrate, then again because Konrad Rudolph, Naveen, Shoban, Daniel Brückner, Yuval F, SilentGhost voted to migrate. – Welbog Sep 1 '09 at 13:26
  • You are right, but what allowed for Rudolph, Naveen, Shoban, Daniel Brückner, Yuval F votes to be counted twice... – perbert Sep 1 '09 at 14:09
  • 2
    I'm guessing a race condition, like I said in the original post. I figure the migration routine checks for >=5 on every vote, rather than ==5. That way any vote 5 or over results in a migration, and a race condition allows there to be more than 5 votes. – Welbog Sep 1 '09 at 14:32

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .