723 Commits

• Merge pull request #15041 from peppy/realm-file-store

  Add realm `FileStore`

  Dan Balasescu · 2021-10-13 13:54:56 +09:00
  df5a76ad9d
• Add a few nullabilities and DCC excludes
  smoogipoo · 2021-10-13 12:51:46 +09:00
  cf10239e70
• Add test coverage of startup import sequence
  Dean Herbert · 2021-10-13 12:19:10 +09:00
  f69a56a26a
• Merge branch 'master' into realm-file-store
  Dean Herbert · 2021-10-12 23:11:02 +09:00
  e468bfb31c
• Merge pull request #15040 from peppy/realm-test-game-host

  Update realm tests to run inside a `GameHost` to allow running on update thread

  Dan Balasescu · 2021-10-12 21:16:06 +09:00
  7bb401b974
• Add basic realm models

  Only the file related ones are really required outside of tests, but
  seems like as good an opportunity as ever to get the rest of the models
  into the game project.

  Dean Herbert · 2021-10-11 15:25:04 +09:00
  6ca415da9f
• Add back update thread verification in RealmContextFactory
  Dean Herbert · 2021-10-11 15:20:12 +09:00
  f43badabf4
• Fix typo in IModelImporter's xmldoc
  Dean Herbert · 2021-10-11 15:12:51 +09:00
  c7675be3ef
• Merge branch 'master' into new-interfaces
  Dan Balasescu · 2021-10-06 11:38:17 +09:00
  4bbff2ebf7
• Merge pull request #14954 from peppy/rename-result-response

  Rename `APIRequest.Result` to `Response`

  Dan Balasescu · 2021-10-06 10:52:33 +09:00
  9491e5a547
• Fix import notifications not showing correct text
  Dean Herbert · 2021-10-05 17:14:09 +09:00
  1e4da81120
• Rename APIRequest.Result to Response
  Dean Herbert · 2021-10-05 14:28:56 +09:00
  b41fa41c85
• Merge branch 'master' into new-interfaces
  Dean Herbert · 2021-10-04 20:42:40 +09:00
  1d99bc280f
• Rename out of place interface name
  Dean Herbert · 2021-10-04 16:35:55 +09:00
  63f0b0c932
• Mark IPresentImports as covariant
  Dean Herbert · 2021-10-04 16:29:46 +09:00
  857000b756
• Remove setter from IHasGuidPrimaryKey interface
  Dean Herbert · 2021-10-04 16:26:28 +09:00
  3c15ef720f
• Merge branch 'master' into importer-returns-live
  Dean Herbert · 2021-10-04 16:24:26 +09:00
  916342c837
• Merge pull request #14901 from peppy/model-importer-split

  Split importer interface out of `IModelManager`

  Dan Balasescu · 2021-10-04 16:22:29 +09:00
  aca2f0c12d
• Merge branch 'master' into realm-context-factory-safer-blocking
  Dean Herbert · 2021-10-02 00:30:09 +09:00
  682fe5be78
• Merge branch 'master' into model-importer-split
  Dean Herbert · 2021-10-02 00:29:29 +09:00
  b1f9bbdc69
• Merge pull request #14900 from peppy/model-downloader-split

  Split `ArchiveModelManager` into two pieces

  Dan Balasescu · 2021-10-01 23:16:09 +09:00
  a620b55d55
• Merge pull request #14906 from peppy/update-realm-context-factory

  Refine `RealmContext` implementation

  Dan Balasescu · 2021-10-01 22:57:03 +09:00
  d24f89fead
• Improve clarity of xmldoc

  Co-authored-by: Dan Balasescu <smoogipoo@smgi.me>

  Dean Herbert · 2021-10-01 22:23:51 +09:00
  a32f5d44e2
• Update all EF based models to implement new read only interfaces
  Dean Herbert · 2021-10-01 16:55:50 +09:00
  d309636460
• Merge branch 'update-realm-context-factory' into realm-context-factory-safer-blocking
  Dean Herbert · 2021-10-01 03:54:31 +09:00
  74841cf1a9
• Guard against disposal in all context retrievals
  Dean Herbert · 2021-10-01 03:46:53 +09:00
  b51fd00ba3
• Add back main context locking
  Dean Herbert · 2021-10-01 03:45:00 +09:00
  8557530cd5
• Ensure realm blocks until all threaded usages are completed
  Dean Herbert · 2021-10-01 01:34:35 +09:00
  cfd3bdf888
• Refine RealmContext implementation API
  Dean Herbert · 2021-09-30 23:56:38 +09:00
  9fa901f6aa
• Initial push to use ILive in import process
  Dean Herbert · 2021-09-30 22:55:25 +09:00
  a2e61883e3
• Remove duplicate interface specification
  Dean Herbert · 2021-09-30 19:25:08 +09:00
  66409147dc
• Split importer interface out of IModelManager
  Dean Herbert · 2021-09-30 18:52:09 +09:00
  c05a8fc4a2
• Split out IModelDownloader and also split apart ScoreManager
  Dean Herbert · 2021-09-30 18:25:20 +09:00
  3e3b9bc963
• Split out IPostNotifications into an interface
  Dean Herbert · 2021-09-30 18:21:34 +09:00
  0a00bc7795
• Expose more pieces of ArchiveModelManager via interfaces
  Dean Herbert · 2021-09-30 16:44:39 +09:00
  5618c9933b
• Move database connection string operations local

  In line with framework changes in
  https://github.com/ppy/osu-framework/pull/4793.

  Dean Herbert · 2021-09-27 17:32:41 +09:00
  d4310f5d9a
• Fix potential nullref throw on failed startup
  Dean Herbert · 2021-09-16 22:48:09 +09:00
  edb1230111
• Remove migration of key bindings
  Dean Herbert · 2021-09-15 17:04:38 +09:00
  5bb741b4e8
• Fix typo in ArchiveModelManager
  Dean Herbert · 2021-08-23 20:23:46 +09:00
  fed0e15cea
• Attempt to reduce skin lookup overhead where file access is not required
  Dean Herbert · 2021-08-17 00:23:30 +09:00
  4bf22db4ff
• Add xmldoc explaining thread safety limitations of IModelManager "events"
  Dean Herbert · 2021-08-10 17:36:59 +09:00
  d9b5f235d8
• Handle nulls and fix missing documentation
  Dean Herbert · 2021-07-05 21:35:37 +09:00
  459f9a0465
• Add ability to lookup multiple users at once to UserLookupCache
  Dean Herbert · 2021-07-05 19:56:16 +09:00
  9c311a6d8a
• Disable foreign key enforcing at an sqlite level
  Dean Herbert · 2021-07-05 15:41:48 +09:00
  1e4beddd2d
• Merge pull request #13767 from peppy/fix-realm-refresh-race

  Fix thread safety of realm `Refresh` operation

  Dan Balasescu · 2021-07-05 13:53:29 +09:00
  b82afaddc1
• Move previousContext assign within lock to make things look safer

  Not an actual requirement, but no harm.

  Dean Herbert · 2021-07-05 10:49:56 +09:00
  f510ef9153
• Mention GetForWrite() as another alternative to Context accesses
  Bartłomiej Dach · 2021-07-04 12:08:15 +02:00
  3291f15dcc
• Mention alternatives to Context when not on update thread in xmldoc
  Bartłomiej Dach · 2021-07-04 12:07:34 +02:00
  b89521314f
• Update tests in line with thread safety check
  Dean Herbert · 2021-07-04 17:59:39 +09:00
  3ec7dc3bb9
• Fix thread safety of realm Refresh operation

  Due to the lack of locking, there was a chance the the update thread
  `context` was retrieved just before the `flushContexts` call, followed
  by `.Refresh()` being run while the blocking behaviour was invoked.
  
  This can be seen in test failures such as
  https://ci.appveyor.com/project/peppy/osu/builds/39859786/tests.
  
  As an aside, I tried multiple different methods to avoid `lock()` on the
  update thread but they felt flaky. The overhead of lock when there's no
  contention is reportedly around 30-50ns, so likely not of concern. We
  can address it at a later point if it becomes one.

  Dean Herbert · 2021-07-04 16:24:43 +09:00
  567e9f33a9