[Asterisk-code-review] res sorcery memory cache: Remove 'prefetch' option. (asterisk[certified/13.1])

Mark Michelson asteriskteam at digium.com
Wed Jul 8 14:37:18 CDT 2015


Mark Michelson has uploaded a new change for review.

  https://gerrit.asterisk.org/834

Change subject: res_sorcery_memory_cache: Remove 'prefetch' option.
......................................................................

res_sorcery_memory_cache: Remove 'prefetch' option.

To prevent confusion I am removing the prefetch option until such
time as it is implemented. All other functionality, however, has
been implemented.

ASTERISK-25067

Change-Id: I9ce6aa3e5c6c5bc3c5baa8ff90fa036d73939895
---
M res/res_sorcery_memory_cache.c
1 file changed, 1 insertion(+), 6 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/34/834/1

diff --git a/res/res_sorcery_memory_cache.c b/res/res_sorcery_memory_cache.c
index 26df12a..dae7606 100644
--- a/res/res_sorcery_memory_cache.c
+++ b/res/res_sorcery_memory_cache.c
@@ -118,8 +118,6 @@
 	unsigned int object_lifetime_maximum;
 	/*! \brief The amount of time (in seconds) before an object is marked as stale, 0 if disabled */
 	unsigned int object_lifetime_stale;
-	/*! \brief Whether objects are prefetched from normal storage at load time, 0 if disabled */
-	unsigned int prefetch;
 	/** \brief Whether all objects are expired when the object type is reloaded, 0 if disabled */
 	unsigned int expire_on_reload;
 	/*! \brief Heap of cached objects. Oldest object is at the top. */
@@ -866,7 +864,7 @@
 
 /*!
  * \internal
- * \brief Callback function to finish configuring the memory cache and to prefetch objects
+ * \brief Callback function to finish configuring the memory cache
  *
  * \param data The sorcery memory cache
  * \param sorcery The sorcery instance
@@ -984,8 +982,6 @@
 					value);
 				return NULL;
 			}
-		} else if (!strcasecmp(name, "prefetch")) {
-			cache->prefetch = ast_true(value);
 		} else if (!strcasecmp(name, "expire_on_reload")) {
 			cache->expire_on_reload = ast_true(value);
 		} else {
@@ -1153,7 +1149,6 @@
 	} else {
 		ast_cli(a->fd, "Object staleness is not enabled - cached objects will not go stale\n");
 	}
-	ast_cli(a->fd, "Prefetch: %s\n", AST_CLI_ONOFF(cache->prefetch));
 	ast_cli(a->fd, "Expire all objects on reload: %s\n", AST_CLI_ONOFF(cache->expire_on_reload));
 
 	ao2_ref(cache, -1);

-- 
To view, visit https://gerrit.asterisk.org/834
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ce6aa3e5c6c5bc3c5baa8ff90fa036d73939895
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: certified/13.1
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list