Skip to content

pack: Check whether inside of coroutine or not #11610

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
cosmo0920 wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

pack: Check whether inside of coroutine or not #11610

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 13 additions & 3 deletions src/flb_pack.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -671,12 +671,22 @@ int flb_pack_json(const char *js, size_t len, char **buffer, size_t *size,
#ifdef FLB_HAVE_SIMD
/*
* When SIMD support is compiled in, route the default JSON pack API through
* the extensible frontend so callers inherit the YYJSON backend selection.
* the extensible frontend so callers inherit the YYJSON backend
* selection.
* However, all of the defaulting to use YYSJON is dangerous on
* coroutine.
* So, we only enabled it for normal thread.
* Still, we use legacy backend on using coroutine.
* Explicit backend-specific entry points remain available for forced JSMN
* or YYJSON behavior.
*/
return flb_pack_json_recs_ext(js, len, buffer, size, root_type,
&records, consumed, NULL);
if (flb_coro_get() != NULL) {
return flb_pack_json_legacy(js, len, buffer, size, root_type, consumed);
}
else {
return flb_pack_json_recs_ext(js, len, buffer, size, root_type,
&records, consumed, NULL);
}
#endif

return flb_pack_json_legacy(js, len, buffer, size, root_type, consumed);
Expand Down
Loading