Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, ccruz, deshani, and 11 others

Git Merge Request #175: Removes neighborhood cache (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Kenton Taylor wants to merge 2 commits from /u/kentontaylor/allura/ to master, 2017-01-09

Commit	Date
[61cfba] (kt/remove_nbhd_cache) by Kenton Taylor fixup! Removes defunct neighborhood cache config entry	2017-01-09 20:27:21	Tree
[775241] by Kenton Taylor Removes neighborhood cache	2017-01-09 16:30:45	Tree

Discussion

Dave Brondsema - 2017-01-09

For some context, the cache has been found to only give about 3ms savings in our tests, and it introduces issues like https://forge-allura.apache.org/p/allura/tickets/7934/ as well as https://forge-allura.apache.org/p/allura/tickets/7890/ in the past.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2017-01-09

Feedback:

neighborhood.cache.duration should be removed from Allura/development.ini
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Kenton Taylor - 2017-01-09

Removed in latest commit.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2017-01-09

Status: open --> merged
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.