SaturninoMateus / jaikuengine

Automatically exported from code.google.com/p/jaikuengine
Apache License 2.0
1 stars 0 forks source link

IM sometimes send out double notifications for posts and comments #78

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The following are examples of notifications that were shown twice in Adium:

* http://jonasl.jaiku.com/presence/4ebf00e0b2b240569db2c7c981454f88
* http://matpe.jaiku.com/presence/0d519b1ef9bc41e5a311ced473520663#c-
65f505eb5ac0469094f25af2194ddd94
* http://tomasw.jaiku.com/presence/55d3a805b9ed4244a2020f2146ba551b#c-
7f55131c163a476f8c6cc88a8a862415

For each pair, both entries were posted within a second of each other.

Original issue reported on code.google.com by jonasnoc...@gmail.com on 1 Apr 2009 at 9:55

GoogleCodeExporter commented 9 years ago
this appears to be related to the new task queue stuff

Original comment by andyster on 2 Apr 2009 at 11:10

GoogleCodeExporter commented 9 years ago
The 2 notifications that I see in Adium seem to be formatted differently. I 
suspect
that one is the html format and one is the plain text version.

Original comment by adewale on 9 Apr 2009 at 10:25

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Some research has yielded that a property of app engine's datastore causes quick
updates to the same entity sometimes to take more time to propagate allowing a 
quick
refetch of it to sometimes get the old data, the approaches to fixing this are 
either:

1. change the architecture such that tasks always delete and write a new task 
with a
different key name
2. add an additional lock in memcache to track progress on tasks

Original comment by andyster on 27 Apr 2009 at 7:43

GoogleCodeExporter commented 9 years ago
This suggests that we load the same property from the datastore multiple times. 
We could also avoid reloading 
multiple times. If this were possible then it would be a simpler solution

Original comment by adewale on 28 Apr 2009 at 12:46

GoogleCodeExporter commented 9 years ago
Issue 124 has been merged into this issue.

Original comment by adewale on 28 Jun 2009 at 10:54