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.

15
  • ... 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 Staff
    Sep 1, 2009 at 11:29
  • Bwahaha! Did you come to the same conclusion that I did?
    – Welbog
    Sep 1, 2009 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, 2009 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 Staff
    Sep 1, 2009 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. Sep 1, 2009 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, 2009 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, 2009 at 11:45
  • Fine; closed and locked... not sure what people will see that is very different, though. Sep 1, 2009 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, 2009 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) Sep 1, 2009 at 12:40
  • That's the spirit!
    – Welbog
    Sep 1, 2009 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, 2009 at 13:00
  • 1
    @Eric: We should make sure that there are no Zombie Questions... Get your close-shotgun handy!
    – perbert
    Sep 1, 2009 at 13:12
  • Again, looks like this one isn't fixed: meta.stackexchange.com/questions/47244/… meta.stackexchange.com/questions/47243/…
    – Welbog
    Apr 20, 2010 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 StaffMod
    May 26, 2010 at 18:20

2 Answers 2

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!

11
  • 1
    Good to know! Thanks for the heads up.
    – Welbog
    Sep 4, 2009 at 1:21
  • 12
    Post it! Post it! Post it! Sep 4, 2009 at 3:19
  • 3
    Alas, programmers everywhere would see the code and then flee our site for another - probably one allowing professional gender reassignment. Sep 4, 2009 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. Sep 4, 2009 at 5:34
  • 1
    xkcd.com/292/- At my last job, doing COBOL, this got taped up to the wall. Sep 4, 2009 at 5:47
  • @AgentConundrum - thank goodness I've got anti-raptor spray! Sep 4, 2009 at 6:28
  • 1
    I can't upvote this. Not with a goto in it! Sep 4, 2009 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. Sep 4, 2009 at 10:26
  • 7
    POST THE CODE, MAN
    – Welbog
    Sep 4, 2009 at 12:50
  • 1
    Hey, this issue is back again: meta.stackexchange.com/questions/47243/… meta.stackexchange.com/questions/47244/…
    – Welbog
    Apr 20, 2010 at 14:24
  • 4
    The goto has been removed - I can sleep again. Jun 1, 2010 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.

3
  • 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, 2009 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, 2009 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, 2009 at 14:32

You must log in to answer this question.

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