in_http: Check the keep_time_key flag only when it's really needed.

Pointed out by repeatedly in the patch review. See the conversation in the following ticket for details: fluent#2020 Signed-off-by: Fujimoto Seiji <[email protected]>
fujimotos · Jun 18, 2018 · 39f3a0d · 39f3a0d
1 parent 6d61cfb
commit 39f3a0d
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/fluent/plugin/in_http.rb b/lib/fluent/plugin/in_http.rb
@@ -205,14 +205,14 @@ def on_request(path_info, params)
 
             if single_record.has_key?('time')
                 single_time = Fluent::EventTime.from_time(Time.at(single_record['time']))
+
+                unless @parser and @parser.keep_time_key
+                    single_record.delete('time')
+                end
             else
                 single_time = time
             end
 
-            unless @parser and @parser.keep_time_key
-                single_record.delete('time')
-            end
-
             mes.add(single_time, single_record)
           end
           router.emit_stream(tag, mes)