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
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
    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 StaffMod
    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 StaffMod
    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 StaffMod
    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 StaffMod
    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
11
  • 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
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 '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 .