activegraph icon indicating copy to clipboard operation
activegraph copied to clipboard

Fix caching bugs in rails reloading

Open yourpalal opened this issue 3 years ago • 2 comments

Add slightly more aggressive clearing of caches during rails reloads.

Fixes query results being wrapped in stale classes after a reload.

This pull introduces/changes:

  • adds ActiveGraph::Node::Labels::Reloading.prepare_for_unload! method
  • call to prepare_for_unload! in an ActiveSupport::Reloader.before_class_unload callback
  • document use of deprecated/removed method
  • replace MODELS_FOR_LABELS_CACHE.clear with ActiveGraph::Node::Labels.clear_wrapped_models, which clears that cache as well as another one!

yourpalal avatar May 27 '21 20:05 yourpalal

Code Climate has analyzed commit 2247b42c and detected 0 issues on this pull request.

View more on Code Climate.

codeclimate[bot] avatar May 27 '21 20:05 codeclimate[bot]

@klobuczek the build errored here due to a failing wget command

The command "wget https://bootstrap.pypa.io/get-pip.py && python3 get-pip.py --user" failed and exited with 1 during .

I'm not sure if this is just a transient issue, but could the build be re-run?

yourpalal avatar Jun 03 '21 15:06 yourpalal