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

Signed-off-by: Shizuo Fujita <[email protected]>
fluent · Jan 31, 2025 · 9b3887f · 9b3887f
1 parent 24f08be
commit 9b3887f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/fluent/plugin/parser_json.rb b/lib/fluent/plugin/parser_json.rb
@@ -52,8 +52,8 @@ 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)
+          log&.info "Oj is not installed, and failing back to JSON for json parser"
+          configure_json_parser(:json)
         when :json then [JSON.method(:load), JSON::ParserError]
         when :yajl then [Yajl.method(:load), Yajl::ParseError]
         else

diff --git a/test/plugin/test_parser_json.rb b/test/plugin/test_parser_json.rb
@@ -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(:load), 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