Pop from queue tail, instead of from head?

[bertspaan]

From @bertspaan to @wires on Slack:

En trouwens!!! Gaat het niet helemaal mis nu vaak omdat we queue omgekeerd lezen, als je dataset update, dat er dan eerst delete- en dan add-berichten op queue komen voor PIT die gewijzigd is? Maar dat core dat dan in omgekeerde volgorde leest?!?!!?!?!?!?!?!?!?

Should we do BRPOP instead? See https://github.com/histograph/core/blob/master/index.js#L45.

[wires]

Signs point to yes 🎱

[tomdemeyer]

Dat verklaart een hoop rare dingen die ik meemaak, af en toe..

Is er iets bekend over oorzaak dat core een restant gequede elementen gewoon niet ziet, soms?
Zou dat iets hiermee te maken kunnen hebben?

Tom Demeyer
Waag Society
http://waag.org

On 13 Jan 2016, at 20:15, Bert Spaan notifications@github.com wrote:

From @bertspaan https://github.com/bertspaan to @wires https://github.com/wires on Slack:

En trouwens!!! Gaat het niet helemaal mis nu vaak omdat we queue omgekeerd lezen, als je dataset update, dat er dan eerst delete- en dan add-berichten op queue komen voor PIT die gewijzigd is? Maar dat core dat dan in omgekeerde volgorde leest?!?!!?!?!?!?!?!?!?

Should we do BRPOP instead? See https://github.com/histograph/core/blob/master/index.js#L45 https://github.com/histograph/core/blob/master/index.js#L45.

—
Reply to this email directly or view it on GitHub #61.

[bertspaan]

Dat heb ik nog nooit gezien... En heeft hier niets mee te maken denk ik. Kun je dat repliceren?

En dit issue is makkelijk te fixen: een R in een L veranderen!

[mmmenno]

Zou #57 hier iets mee te maken hebben? Die niet weg te krijgen rijksmonumenten dataset met foute id's is mij een doorn in het oog.

[jobspierings]

Issue not clear: what's the problem and how to check whether it is fixed?

[bertspaan]

Issue quite clear.

[sbocconi]

The problem is clear, only it would help to have a way to replicate it to see when it is fixed.

[wires]

I'll push a fix today

[bertspaan]

👯