Open
Conversation
Member
Author
|
@sevenc-nanashi 何かご意見があれば大変助かります。 (とりあえずWindowsにbuild/Packagesという大きめなやつが見えるので、これも対象にしようかなと思ってます) |
Hiroshiba
reviewed
Feb 11, 2026
Member
There was a problem hiding this comment.
一般的な意見だけ書きます!
こうしてほしいというより、こういう観点があるよというニュアンスです。
知っていると思いますが、基本的にキャッシュは危ないんですよね。
スコープをまたいで未来方向全部にグローバルな情報になるので。
キャッシュ書き換えるコードも、将来これを忘れてバグが生まれちゃうはず。
なので無くて良いなら無い方が理想ではある。
最高の案は思いついてないのですが、案まで:
目的はたぶん「失敗したものの再挑戦をすぐ行えること」ですよね。
なのでfailしたときにだけキャッシュをアップロードするのはどうでしょう。
で、引数指定でどのリビルドをやり直すか指定できるようにするとか。
まあコード変更が大変ですが・・・。
あるいはビルド産物からリリースへの加工はOS非依存でしょうか。
であればcacheじゃなくてartifactにアップロードして、ビルド産物の加工はローカルPCで別スクリプト書いて実行し、そのスクリプトをリポジトリにcommitしてメンテするとかもありそう。
とはいえ諸刃の剣だとしてもキャッシュが便利だというのは理解できるので、全然強い意見じゃないです!
キャッシュやキャッシュ加工は危ないよという一般論と、別アイデアの助けになればくらいの気持ちでコメントしました。
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
内容
WebGPU版ビルドが入った #90 以降キャッシュの合計が10GBを超過してしまい、キャッシュ機構があまり機能しなくなってしまった問題があったのでそれに対処する。
その他
このPRを作った段階で、以下の状態までは削減できている。