From cc9a11346bf53dccade7aa3aae70f7514ad9b838 Mon Sep 17 00:00:00 2001 From: Jochen Schalanda Date: Fri, 17 Jul 2015 10:48:33 +0200 Subject: [PATCH] Avoid repeated hash lookup for StreamRule#getType() --- .../main/java/org/graylog2/streams/StreamRouterEngine.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/graylog2-server/src/main/java/org/graylog2/streams/StreamRouterEngine.java b/graylog2-server/src/main/java/org/graylog2/streams/StreamRouterEngine.java index f2b59dd1ea04..67a1274cd239 100644 --- a/graylog2-server/src/main/java/org/graylog2/streams/StreamRouterEngine.java +++ b/graylog2-server/src/main/java/org/graylog2/streams/StreamRouterEngine.java @@ -149,23 +149,24 @@ public List match(Message message) { final Set blackList = Sets.newHashSet(); for (final Rule rule : rulesList) { - final Stream.MatchingType matchingType = rule.getMatchingType(); if (blackList.contains(rule.getStream())) { continue; } final StreamRule streamRule = rule.getStreamRule(); - if (streamRule.getType() != StreamRuleType.PRESENCE && !message.hasField(streamRule.getField())) { + final StreamRuleType streamRuleType = streamRule.getType(); + if (streamRuleType != StreamRuleType.PRESENCE && !message.hasField(streamRule.getField())) { continue; } final Stream stream; - if (streamRule.getType() != StreamRuleType.REGEX) { + if (streamRuleType != StreamRuleType.REGEX) { stream = rule.match(message); } else { stream = matchWithTimeOut(message, rule); } + final Stream.MatchingType matchingType = rule.getMatchingType(); if (stream == null) { if (matchingType == Stream.MatchingType.AND) { result.remove(rule.getStream());