code review

cat-consumer/src/main/java/com/dianping/cat/consumer/failure/FailureReportAnalyzer.java

@@ -179,8 +179,8 @@ public class FailureReportAnalyzer extends AbstractMessageAnalyzer<FailureReport
 			}
 		}
 		oldSegments.clear();
-		for (String key : newSegments.keySet()) {
-			oldSegments.put(key, newSegments.get(key));
+		for (java.util.Map.Entry<String, Segment> entry : newSegments.entrySet()) {
+			oldSegments.put(entry.getKey(), newSegments.get(entry.getValue()));
 		}
 		return m_report;
 	}

cat-home/src/main/java/com/dianping/cat/report/ReportManager.java

@@ -47,8 +47,6 @@ public abstract class ReportManager {
 			} catch (ParseException e) {
 				e.printStackTrace();
 			}
-		} else {
-			inputStart = DateUtils.SDF_URL.format(currentHourStart);
 		}
 
 		long reportStart = startLong + changeValue * DateUtils.HOUR;

cat-home/src/main/java/com/dianping/cat/report/page/service/provider/TransactionModelProvider.java

@@ -49,8 +49,13 @@ public class TransactionModelProvider implements ModelProvider {
 		} else if (index.equals(Constants.MEMORY_LAST)) {
 			analyzer = (TransactionReportAnalyzer) m_consumer.getLastAnalyzer("transaction");
 		}
+		TransactionReport report;
 		
-		TransactionReport report = analyzer.generate(domain);
+		if (analyzer == null) {
+			report = new TransactionReport(domain);
+		} else {
+			report = analyzer.generate(domain);
+		}
 		return new DefaultXmlBuilder().buildXml(report);
 	}