search

histograph / core

Histograph Core: Graph management and inferencing
MIT License
1 stars 3 forks source link

Pop from queue tail, instead of from head? #61

Open bertspaan opened 8 years ago

bertspaan commented 8 years ago

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 commented 8 years ago

Signs point to yes :8ball:

tomdemeyer commented 8 years ago

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 https://github.com/histograph/core/issues/61.

bertspaan commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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

bertspaan commented 8 years ago

Issue quite clear.

sbocconi commented 8 years ago

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

wires commented 8 years ago

I'll push a fix today

bertspaan commented 8 years ago

:dancers: