ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
7,068 Commits 324 Branches 20,019 Tags 1.7 GiB
Commit Graph

4 Commits

Author SHA1 Message Date
Brian Palmer 52b4ddc57a add attr_accessible to EventStream::Failure model 
test plan: spec/lib/event_stream/** specs should still pass

Change-Id: I02418dde483ff7824bff2ee0eb088295422bb415
Reviewed-on: https://gerrit.instructure.com/21347
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
Product-Review: Brian Palmer <brianp@instructure.com>
QA-Review: Brian Palmer <brianp@instructure.com>
 2013-06-11 14:56:09 +00:00
Cody Cutrer 2ac4618fab activate the user's shard when consulting cassandra 
refs CNVS-6018

test plan:
 * saving and loading of cassandra page views should work

Change-Id: Ic9730371bb1f4207cb06aa6da30baee9f4232d8f
Reviewed-on: https://gerrit.instructure.com/20979
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
QA-Review: Brian Palmer <brianp@instructure.com>
Product-Review: Cody Cutrer <cody@instructure.com>
 2013-06-07 16:53:22 +00:00
Jacob Fugal 261e983f61 capture failed EventStream operations in DB 
Create an event_stream_failures table, and add to it when an EventStream
operation intended for cassandra fails. intentionally swallows the
error, so the code inserting/updating the event stream doesn't notice
and bail on the failure, but we can keep an eye our for new rows and
also have enough data to replay those failures once fixed.

fixes CNVS-4161

test-plan:
 - enable cassandra page views but intentionally misconfigure so that
   inserts into cassandra fail
 - generate traffic in canvas that should create page views
   - the browsing user should not notice any errors
   - but page views should stay empty
   - event_stream_failures should be getting new rows

Change-Id: I1acabfef785ee0637b241924c6ff254c41d3d1e3
Reviewed-on: https://gerrit.instructure.com/19975
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
QA-Review: Clare Strong <clare@instructure.com>
Product-Review: Jacob Fugal <jacob@instructure.com>
 2013-04-30 15:04:16 +00:00
Jacob Fugal 45aec302c4 EventStream abstraction 
fixes CNVS-4159

Change-Id: I8a1ce45bd8d50bd767dfaaa6ca51dff6a0bdffd5
Reviewed-on: https://gerrit.instructure.com/19103
QA-Review: Clare Strong <clare@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
Tested-by: Brian Palmer <brianp@instructure.com>
Product-Review: Jacob Fugal <jacob@instructure.com>
 2013-04-19 19:05:14 +00:00