nic-6443 commented on code in PR #13386:
URL: https://github.com/apache/apisix/pull/13386#discussion_r3264810008
##########
apisix/cli/config.lua:
##########
@@ -29,6 +29,7 @@ local _M = {
enable_server_tokens = true,
extra_lua_path = "",
extra_lua_cpath = "",
+ request_body_json_lib = "qjson",
Review Comment:
I agree with this concern. I changed the default to `simdjson` in
`129d57522`, and `qjson` is now documented as an experimental option instead of
the default path.
The benchmark still shows qjson has strong throughput for large AI request
bodies, but it is better to let users opt in explicitly after evaluating
compatibility for their workloads. The default should be the more conservative
accelerated decoder for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
