parser_json: use JSON as fallback parser instead of Yajl for performance

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>

Author: Watson1978

lib/fluent/plugin/parser_json.rb

@@ -52,9 +52,9 @@ def configure_json_parser(name)
         when :oj
           return [Oj.method(:load), Oj::ParseError] if Fluent::OjOptions.available?
 
-          log&.info "Oj is not installed, and failing back to Yajl for json parser"
-          configure_json_parser(:yajl)
-        when :json then [JSON.method(:load), JSON::ParserError]
+          log&.info "Oj is not installed, and failing back to JSON for json parser"
+          configure_json_parser(:json)
+        when :json then [JSON.method(:parse), JSON::ParserError]
         when :yajl then [Yajl.method(:load), Yajl::ParseError]
         else
           raise "BUG: unknown json parser specified: #{name}"

test/plugin/test_parser_json.rb

@@ -10,7 +10,7 @@ def setup
 
   sub_test_case "configure_json_parser" do
     data("oj", [:oj, [Oj.method(:load), Oj::ParseError]])
-    data("json", [:json, [JSON.method(:load), JSON::ParserError]])
+    data("json", [:json, [JSON.method(:parse), JSON::ParserError]])
     data("yajl", [:yajl, [Yajl.method(:load), Yajl::ParseError]])
     def test_return_each_loader((input, expected_return))
       result = @parser.instance.configure_json_parser(input)
@@ -28,12 +28,12 @@ def test_fall_back_oj_to_yajl_if_oj_not_available
 
       result = @parser.instance.configure_json_parser(:oj)
 
-      assert_equal [Yajl.method(:load), Yajl::ParseError], result
+      assert_equal [JSON.method(:parse), JSON::ParserError], result
       logs = @parser.logs.collect do |log|
         log.gsub(/\A\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-+]\d{4} /, "")
       end
       assert_equal(
-        ["[info]: Oj is not installed, and failing back to Yajl for json parser\n"],
+        ["[info]: Oj is not installed, and failing back to JSON for json parser\n"],
         logs
       )
     end