You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2010/09/22 21:57:24 UTC

svn commit: r1000161 - /trafficserver/traffic/trunk/iocore/cache/Cache.cc

Author: jplevyak
Date: Wed Sep 22 19:57:23 2010
New Revision: 1000161

URL: http://svn.apache.org/viewvc?rev=1000161&view=rev
Log:
TS-453: fix ram cache hit/miss stat duplication by removing them from Cache.cc

Modified:
    trafficserver/traffic/trunk/iocore/cache/Cache.cc

Modified: trafficserver/traffic/trunk/iocore/cache/Cache.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/iocore/cache/Cache.cc?rev=1000161&r1=1000160&r2=1000161&view=diff
==============================================================================
--- trafficserver/traffic/trunk/iocore/cache/Cache.cc (original)
+++ trafficserver/traffic/trunk/iocore/cache/Cache.cc Wed Sep 22 19:57:23 2010
@@ -1900,20 +1900,18 @@ CacheVC::handleRead(int event, Event *e)
   cancel_trigger();
 
   f.doc_from_ram_cache = false;
+
   // check ram cache
   ink_debug_assert(part->mutex->thread_holding == this_ethread());
-  if (part->ram_cache->get(read_key, &buf, 0, dir_offset(&dir))) {
-    CACHE_INCREMENT_DYN_STAT(cache_ram_cache_hits_stat);
+  if (part->ram_cache->get(read_key, &buf, 0, dir_offset(&dir)))
     goto LramHit;
-  }
+
   // check if it was read in the last open_read call
   if (*read_key == part->first_fragment_key && dir_offset(&dir) == part->first_fragment_offset) {
     buf = part->first_fragment_data;
     goto LmemHit;
   }
 
-  CACHE_INCREMENT_DYN_STAT(cache_ram_cache_misses_stat);
-
   // see if its in the aggregation buffer
   if (dir_agg_buf_valid(part, &dir)) {
     int agg_offset = part_offset(part, &dir) - part->header->write_pos;