Skip to content

Commit

Permalink
fix: persistent cache remove non-existent modules from make_failed_mo…
Browse files Browse the repository at this point in the history
…dule (#9309)

fix: cache remove non-existent modules from make_failed_module
  • Loading branch information
jerrykingxyz authored Feb 14, 2025
1 parent b7b7c3e commit f34377a
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions crates/rspack_core/src/cache/persistent/occasion/make/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,12 @@ impl MakeOccasion {
artifact.build_dependencies = build_dep;
artifact.reset_dependencies_incremental_info();

// TODO remove it after all of module are cacheable
let mut make_failed_module = std::mem::take(&mut artifact.make_failed_module);
let mg = artifact.get_module_graph_mut();
make_failed_module.retain(|module_id| mg.module_by_identifier(module_id).is_some());
artifact.make_failed_module = make_failed_module;

Ok(artifact)
}
}

2 comments on commit f34377a

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented on f34377a Feb 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 Ecosystem CI detail: Open

suite result
modernjs ✅ success
rspress ✅ success
rslib ✅ success
rsbuild ❌ failure
rsdoctor ❌ failure
examples ✅ success
devserver ✅ success
nuxt ✅ success

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented on f34377a Feb 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 Benchmark detail: Open

Name Base (2025-02-14 05524cd) Current Change
10000_big_production-mode_disable-minimize + exec 37.9 s ± 263 ms 39.9 s ± 754 ms +5.40 %
10000_development-mode + exec 1.83 s ± 21 ms 1.87 s ± 35 ms +1.78 %
10000_development-mode_hmr + exec 680 ms ± 7.1 ms 697 ms ± 26 ms +2.47 %
10000_production-mode + exec 2.29 s ± 68 ms 2.31 s ± 33 ms +1.05 %
10000_production-mode_persistent-cold + exec 2.44 s ± 66 ms 2.49 s ± 75 ms +2.02 %
10000_production-mode_persistent-hot + exec 1.68 s ± 135 ms 1.66 s ± 44 ms -1.16 %
arco-pro_development-mode + exec 1.8 s ± 164 ms 1.79 s ± 173 ms -0.66 %
arco-pro_development-mode_hmr + exec 388 ms ± 1.9 ms 388 ms ± 2.6 ms -0.03 %
arco-pro_production-mode + exec 3.6 s ± 123 ms 3.67 s ± 249 ms +1.93 %
arco-pro_production-mode_generate-package-json-webpack-plugin + exec 3.63 s ± 134 ms 3.69 s ± 82 ms +1.49 %
arco-pro_production-mode_persistent-cold + exec 3.79 s ± 176 ms 3.79 s ± 103 ms +0.15 %
arco-pro_production-mode_persistent-hot + exec 2.37 s ± 75 ms 2.42 s ± 117 ms +2.17 %
arco-pro_production-mode_traverse-chunk-modules + exec 3.62 s ± 183 ms 3.65 s ± 121 ms +0.86 %
large-dyn-imports_development-mode + exec 2.12 s ± 18 ms 2.17 s ± 153 ms +2.70 %
large-dyn-imports_production-mode + exec 2.15 s ± 48 ms 2.19 s ± 29 ms +1.60 %
threejs_development-mode_10x + exec 1.58 s ± 69 ms 1.55 s ± 26 ms -1.93 %
threejs_development-mode_10x_hmr + exec 785 ms ± 7.9 ms 771 ms ± 5.7 ms -1.75 %
threejs_production-mode_10x + exec 5.25 s ± 50 ms 5.28 s ± 91 ms +0.57 %
threejs_production-mode_10x_persistent-cold + exec 5.31 s ± 95 ms 5.34 s ± 130 ms +0.52 %
threejs_production-mode_10x_persistent-hot + exec 4.52 s ± 102 ms 4.54 s ± 155 ms +0.43 %
10000_big_production-mode_disable-minimize + rss memory 8704 MiB ± 49.5 MiB 8708 MiB ± 105 MiB +0.04 %
10000_development-mode + rss memory 636 MiB ± 27.9 MiB 647 MiB ± 25.8 MiB +1.65 %
10000_development-mode_hmr + rss memory 1223 MiB ± 158 MiB 1290 MiB ± 134 MiB +5.52 %
10000_production-mode + rss memory 628 MiB ± 14.1 MiB 623 MiB ± 27.9 MiB -0.84 %
10000_production-mode_persistent-cold + rss memory 735 MiB ± 12.5 MiB 737 MiB ± 29.8 MiB +0.36 %
10000_production-mode_persistent-hot + rss memory 719 MiB ± 3.87 MiB 715 MiB ± 21.1 MiB -0.53 %
arco-pro_development-mode + rss memory 583 MiB ± 24.4 MiB 572 MiB ± 13.4 MiB -1.90 %
arco-pro_development-mode_hmr + rss memory 644 MiB ± 77 MiB 646 MiB ± 58.1 MiB +0.22 %
arco-pro_production-mode + rss memory 724 MiB ± 7.62 MiB 707 MiB ± 9.44 MiB -2.37 %
arco-pro_production-mode_generate-package-json-webpack-plugin + rss memory 728 MiB ± 15.8 MiB 727 MiB ± 8.47 MiB -0.15 %
arco-pro_production-mode_persistent-cold + rss memory 843 MiB ± 21.6 MiB 842 MiB ± 37.5 MiB -0.12 %
arco-pro_production-mode_persistent-hot + rss memory 710 MiB ± 22.1 MiB 702 MiB ± 32.8 MiB -1.14 %
arco-pro_production-mode_traverse-chunk-modules + rss memory 709 MiB ± 8.18 MiB 719 MiB ± 22 MiB +1.30 %
large-dyn-imports_development-mode + rss memory 643 MiB ± 3.77 MiB 637 MiB ± 11.1 MiB -0.86 %
large-dyn-imports_production-mode + rss memory 525 MiB ± 6.6 MiB 525 MiB ± 4.2 MiB -0.05 %
threejs_development-mode_10x + rss memory 552 MiB ± 18.3 MiB 547 MiB ± 25.4 MiB -0.99 %
threejs_development-mode_10x_hmr + rss memory 1105 MiB ± 90.1 MiB 1150 MiB ± 65.4 MiB +4.04 %
threejs_production-mode_10x + rss memory 841 MiB ± 33 MiB 835 MiB ± 45 MiB -0.70 %
threejs_production-mode_10x_persistent-cold + rss memory 968 MiB ± 36.3 MiB 947 MiB ± 55.8 MiB -2.21 %
threejs_production-mode_10x_persistent-hot + rss memory 866 MiB ± 43.1 MiB 869 MiB ± 23.2 MiB +0.39 %

Threshold exceeded: ["10000_big_production-mode_disable-minimize + exec"]

Please sign in to comment.