Skip to content

Commit

Permalink
Merge pull request #1224 from mebigfatguy/master
Browse files Browse the repository at this point in the history
remove dead field defaultNameCache
  • Loading branch information
mattrjacobs committed May 26, 2016
2 parents 8414a4e + 45b1de0 commit d85daae
Showing 1 changed file with 0 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,6 @@ private RequestCollapser<BatchReturnType, ResponseType, RequestArgumentType> get
* Clears all state. If new requests come in instances will be recreated and metrics started from scratch.
*/
public static void reset() {
defaultNameCache.clear();
globalScopedCollapsers.clear();
requestScopedCollapsers.clear();
HystrixTimer.reset();
Expand Down Expand Up @@ -283,10 +282,4 @@ public Setter andCollapserPropertiesDefaults(HystrixCollapserProperties.Setter p
}

}

// this is a micro-optimization but saves about 1-2microseconds (on 2011 MacBook Pro)
// on the repetitive string processing that will occur on the same classes over and over again
@SuppressWarnings("rawtypes")
private static ConcurrentHashMap<Class<? extends HystrixCollapser>, String> defaultNameCache = new ConcurrentHashMap<Class<? extends HystrixCollapser>, String>();

}

0 comments on commit d85daae

Please sign in to comment.