Make the trooper queue update more quickly |
||
Issue descriptionAs a trooper, I think it is a little jarring to update an issue then have to wait a minute before the issue disappears from Sheriff-o-Matic. I think this is more jarring for troopers than for sheriffs since troopers work more directly with bugs. AppEngine doesn't let you run crons more often than once a minute. So I guess we'd have to come up with something else. We could just not cache the trooper queue, but then it takes longer to load and will probably error more often. One possible solution though it might come across as really janky is that since we are currently already loading trooper bugs in two parts (first part cached bugs assigned to every trooper, second part, bugs assigned to you), perhaps we could make the second slower read return the whole uncached trooper queue and replace the bugs when it finishes. ie: trooper loads page and immediately gets queue populated with data that might be stale/incomplete, trooper queue shows that it is still loading though, and updates with latest most up to date data after ~5 seconds.
,
Jan 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/b04c44bf6ebe42624eec01228d824cfed2e5a54d commit b04c44bf6ebe42624eec01228d824cfed2e5a54d Author: Tiff Zhang <zhangtiff@google.com> Date: Tue Jan 24 21:06:53 2017 SoM: Adjust trooper queue to load unstale data after loading cached data. BUG= 683253 Change-Id: I4a489fb7309628801bb1f6b7cc2fb59f99d34a38 Reviewed-on: https://chromium-review.googlesource.com/431839 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org> [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/elements/som-bug-queue/som-bug-queue.html [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/elements/som-bug-queue/som-bug-queue.js [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/som/bugqueue.go [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/som/main.go [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/som/main_test.go [modify] https://crrev.com/b04c44bf6ebe42624eec01228d824cfed2e5a54d/go/src/infra/appengine/sheriff-o-matic/test/som-bug-queue-test.html
,
Jan 24 2017
This will be deployed later today. |
||
►
Sign in to add a comment |
||
Comment 1 by zhangtiff@chromium.org
, Jan 20 2017