modify the code name

cat-consumer-advanced/src/main/resources/META-INF/plexus/components.xml

@@ -138,6 +138,9 @@
 				<requirement>
 					<role>com.dianping.cat.consumer.core.ProductLineConfigManager</role>
 				</requirement>
+				<requirement>
+					<role>com.dianping.cat.abtest.spi.internal.ABTestCodec</role>
+				</requirement>
 			</requirements>
 		</component>
 		<component>

cat-core/src/main/java/com/dianping/cat/abtest/spi/internal/DefaultABTestCodec.java

@@ -7,6 +7,15 @@ import java.util.Set;
 
 public class DefaultABTestCodec implements ABTestCodec {
 
+	private Map<String, Map<String, String>> m_codes = new LinkedHashMap<String, Map<String, String>>(1000, 0.75f, true) {
+		private static final long serialVersionUID = 1L;
+
+		@Override
+		protected boolean removeEldestEntry(Entry<String, Map<String, String>> arg0) {
+			return true;
+		}
+	};
+
 	public Map<String, Map<String, String>> decode(String value, Set<String> keys) {
 		int len = value.length();
 		Map<String, Map<String, String>> map = new LinkedHashMap<String, Map<String, String>>();
@@ -129,21 +138,26 @@ public class DefaultABTestCodec implements ABTestCodec {
 		return sb.toString();
 	}
 
-	public Map<String,String> decode(String value){
-		Map<String, Map<String, String>> maps = decode(value, null) ;
-		Map<String,String> map = new LinkedHashMap<String, String>();
-		
-		for( Entry<String, Map<String, String>> entry : maps.entrySet()){
-			String key = entry.getKey();
-			if(entry.getValue() != null){
-				String val = entry.getValue().get("ab");
-				
-				if(val != null){
-					map.put(key, val);
+	public Map<String, String> decode(String value) {
+		Map<String, String> code = m_codes.get(value);
+
+		if (code == null) {
+			Map<String, Map<String, String>> maps = decode(value, null);
+
+			code = new LinkedHashMap<String, String>();
+
+			for (Entry<String, Map<String, String>> entry : maps.entrySet()) {
+				String key = entry.getKey();
+				if (entry.getValue() != null) {
+					String val = entry.getValue().get("ab");
+
+					if (val != null) {
+						code.put(key, val);
+					}
 				}
 			}
+			m_codes.put(value, code);
 		}
-		
-		return map;
+		return code;
 	}
 }

cat-home/src/main/java/com/dianping/cat/report/page/metric/Handler.java

@@ -69,7 +69,7 @@ public class Handler implements PageHandler<Context> {
 		normalize(model, payload);
 
 		MetricReport report = getReport(payload);
-		int test = payload.getTest();
+		String test = payload.getTest();
 
 		if (report != null) {
 			Date startTime = report.getStartTime();

cat-home/src/main/java/com/dianping/cat/report/page/metric/MetricDisplay.java

@@ -22,9 +22,9 @@ public class MetricDisplay extends BaseVisitor {
 
 	private Map<String, LineChart> m_lineCharts = new LinkedHashMap<String, LineChart>();
 
-	private Set<Integer> m_abtests = new TreeSet<Integer>();
+	private Set<String> m_abtests = new TreeSet<String>();
 
-	private int m_abtest;
+	private String m_abtest;
 
 	private Date m_start;
 
@@ -40,11 +40,11 @@ public class MetricDisplay extends BaseVisitor {
 		return new ArrayList<LineChart>(m_lineCharts.values());
 	}
 	
-	public Set<Integer> getAbtests(){
+	public Set<String> getAbtests(){
 		return m_abtests;
 	}
 
-	public MetricDisplay(List<BusinessConfig> configs, int abtest, Date start) {
+	public MetricDisplay(List<BusinessConfig> configs, String abtest, Date start) {
 		m_start = start;
 		m_abtest = abtest;
 
@@ -80,10 +80,10 @@ public class MetricDisplay extends BaseVisitor {
 
 	@Override
 	public void visitAbtest(Abtest abtest) {
-		Integer abtestId = abtest.getRunId();
+		String abtestId = abtest.getRunId();
 
 		m_abtests.add(abtestId);
-		if (m_abtest == abtestId) {
+		if (m_abtest.equals(abtestId)) {
 			super.visitAbtest(abtest);
 		}
 	}

cat-home/src/main/java/com/dianping/cat/report/page/metric/Payload.java

@@ -16,7 +16,7 @@ public class Payload extends AbstractReportPayload<Action> {
 	private String m_product;
 
 	@FieldMeta("test")
-	private int m_test = -1;
+	private String m_test = "-1";
 
 	public Payload() {
 		super(ReportPage.METRIC);
@@ -42,20 +42,20 @@ public class Payload extends AbstractReportPayload<Action> {
 	}
 
 	public String getProduct() {
-   	return m_product;
-   }
+		return m_product;
+	}
 
 	public void setProduct(String product) {
-   	m_product = product;
-   }
+		m_product = product;
+	}
 
-	public int getTest() {
-   	return m_test;
-   }
+	public String getTest() {
+		return m_test;
+	}
 
-	public void setTest(int test) {
-   	m_test = test;
-   }
+	public void setTest(String test) {
+		m_test = test;
+	}
 
 	@Override
 	public void validate(ActionContext<?> ctx) {