Forum OpenACS Q&A: Re: Clustering parameter caching issue

Collapse
Posted by Jonathan Kelley on
Hi Gustaf,

I tried applying the patch in this commit to an existing 5.10.1b4 version of the acs-tcl package and found that it was not updating the cache properly for a custom package parameter.

Here is the commit I patched:
https://github.com/openacs/openacs-core/commit/c0a1cf7b98866f00608634b4df718c754f89c784

I don't see any errors in the log files and it appears to propagate the message to the cluster nodes but even the node that the parameter was changed on didn't update the cache.

If I modify the code only slightly the cache is updated properly. Here is the minor update I made along the same lines as Khy's initial submission.

@@ -515,9 +515,7 @@ if {[ns_config "ns/parameters" cachingmode "per-node"] eq "none"} {

} {
if {$delete_p} {
- if {[nsv_dict exists ad_param $key $parameter_name]} {
- ::acs::clusterwide nsv_dict unset ad_param $key $parameter_name
- }
+ ::acs::clusterwide nsv_dict unset ad_param $key $parameter_name
acs::per_request_cache flush -pattern acs-tcl.ad_param-$key
return
}

Any suggestions?