ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

#684 closed defect (fixed)

ファンクションキャッシュ有効の場合にナビゲーションの変更反映が遅延

Reported by: ogawa Owned by: ebihara
Priority: critical Milestone: OpenPNE2.6.7
Component: core【OpenPNEコア部分】 Version:
Keywords: Cc:

Description

fetch_inc_navi() のファンクションキャッシュをちゃんとドロップできていないのが問題。

  • update_skin_image スキン画像変更の際にドロップしていない
  • update_c_navi でtarget_idなしのドロップしかしていないので、h_系以外はちゃんとドロップできていない

Change History (15)

comment:1 Changed 12 years ago by ebihara

Owner: changed from ogawa to ebihara
Status: newassigned

やります。

comment:2 Changed 12 years ago by ebihara

  • h_系についてはupdate_skin_imageでドロップするようにする
  • f_系とc_系についてはドロップができないので、キャッシュ生存時間を短くすることで対処

以上のようにして修正したいと思います。

comment:3 Changed 12 years ago by ebihara

以下のリビジョンで、個人ブランチ上のコードに対して修正しました。

マージしてみて問題がなさそうであれば、リリースブランチにコミットします。

comment:4 Changed 12 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンでリリースブランチにコミットしました。ご確認ください。

comment:5 Changed 12 years ago by ogawa

Status: newassigned

確認します。

comment:6 Changed 12 years ago by ogawa

Owner: changed from ogawa to ebihara
Status: assignednew
  • h_系のみキャッシュする
  • f_系、c_系はキャッシュしない

としてください。5分間もナビが表示されないのは許されません。

また、fetch_inc_navi()内で呼んでいる util_get_c_navi() の内容もキャッシュするので、管理画面のナビゲーションのリンク先変更のアクションでもキャッシュドロップの処理を加えてください。

comment:7 Changed 12 years ago by ebihara

Status: newassigned

了解です。対応します。

comment:8 Changed 12 years ago by ebihara

また、fetch_inc_navi()内で呼んでいる util_get_c_navi() の内容もキャッシュするので、管理画面のナビゲーションのリンク先変更のアクションでもキャッシュドロップの処理を加えてください。

リンク先の変更を行う admin_do_update_c_navi 内ではキャッシュドロップ処理を行っていましたが、変更した設定を元に戻す admin_do_delete_c_navi ではドロップ処理を行っていませんでしたので、遅延が発生していました。

これについては、 admin_do_delete_c_navi 内に admin_do_update_c_navi のようなドロップ処理を記述することで対応します。

comment:9 Changed 12 years ago by ebihara

以下のリビジョンで個人用ブランチにコミットしました。マージしてみて問題がなければ、リリースブランチにコミットします。

comment:10 Changed 12 years ago by ebihara

以下のリビジョンで、fetch_inc_navi()のキャッシュ生存期間をFASTからLONGに戻しました。これもマージしてみて問題がなければ、あわせてリリースブランチにコミットします。

comment:11 Changed 12 years ago by ebihara

以下のリビジョンで、リリースブランチにコミットしました。ご確認よろしくお願いします。

comment:12 Changed 12 years ago by ebihara

Owner: changed from ebihara to ogawa
Status: assignednew

comment:13 Changed 12 years ago by ogawa

Status: newassigned

確認します。

comment:14 Changed 12 years ago by ogawa

Owner: changed from ogawa to ebihara
Status: assignednew

comment:15 Changed 12 years ago by ogawa

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.