diff --git a/cat-core/src/main/java/com/dianping/cat/config/server/ServerConfigManager.java b/cat-core/src/main/java/com/dianping/cat/config/server/ServerConfigManager.java index a9755157b0495499d3bee468b6739c213073e080..149153be6aca340689890416fd1e02f666b0fc40 100644 --- a/cat-core/src/main/java/com/dianping/cat/config/server/ServerConfigManager.java +++ b/cat-core/src/main/java/com/dianping/cat/config/server/ServerConfigManager.java @@ -43,7 +43,7 @@ public class ServerConfigManager implements Initializable, LogEnabled { private Set m_unusedNames = new HashSet(); - private Set m_crashLogs = new HashSet(); + private Set m_crashLogDomains = new HashSet(); private Set m_invalidateDomains = new HashSet(); @@ -107,7 +107,7 @@ public class ServerConfigManager implements Initializable, LogEnabled { } public Set getCrashLogs() { - return m_crashLogs; + return m_crashLogDomains; } public String getHdfsBaseDir(String id) { @@ -251,7 +251,11 @@ public class ServerConfigManager implements Initializable, LogEnabled { } public Set getUnusedDomains() { - return m_invalidateDomains; + Set unusedDomains = new HashSet(); + + unusedDomains.addAll(m_invalidateDomains); + unusedDomains.addAll(m_crashLogDomains); + return unusedDomains; } @Override @@ -278,19 +282,13 @@ public class ServerConfigManager implements Initializable, LogEnabled { m_invalidateDomains.add(Constants.FRONT_END); m_invalidateDomains.add("paas"); m_invalidateDomains.add("SMS-RECEIVER"); - m_invalidateDomains.add("AndroidCrashLog"); - m_invalidateDomains.add("iOSCrashLog"); - m_invalidateDomains.add("MerchantAndroidCrashLog"); - m_invalidateDomains.add("MerchantIOSCrashLog"); - m_invalidateDomains.add("ApolloAndroidCrashLog"); - m_invalidateDomains.add("ApolloIOSCrashLog"); - m_crashLogs.add("AndroidCrashLog"); - m_crashLogs.add("iOSCrashLog"); - m_crashLogs.add("MerchantAndroidCrashLog"); - m_crashLogs.add("MerchantIOSCrashLog"); - m_crashLogs.add("ApolloAndroidCrashLog"); - m_crashLogs.add("ApolloIOSCrashLog"); + m_crashLogDomains.add("AndroidCrashLog"); + m_crashLogDomains.add("iOSCrashLog"); + m_crashLogDomains.add("MerchantAndroidCrashLog"); + m_crashLogDomains.add("MerchantIOSCrashLog"); + m_crashLogDomains.add("ApolloAndroidCrashLog"); + m_crashLogDomains.add("ApolloIOSCrashLog"); } public void initialize(File configFile) throws Exception { @@ -344,7 +342,7 @@ public class ServerConfigManager implements Initializable, LogEnabled { } public boolean isCrashLog(String domain) { - return m_crashLogs.contains(domain); + return m_crashLogDomains.contains(domain); } public boolean isHdfsOn() { @@ -415,7 +413,8 @@ public class ServerConfigManager implements Initializable, LogEnabled { } public boolean validateDomain(String domain) { - return !m_invalidateDomains.contains(domain) && StringUtils.isNotEmpty(domain); + return !m_invalidateDomains.contains(domain) && !m_crashLogDomains.contains(domain) + && StringUtils.isNotEmpty(domain); } public boolean validateIp(String str) {