Skip to content

gh-116738: Make _csv module thread-safe #118344

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

Merged
merged 17 commits into from
Oct 11, 2024
Merged

gh-116738: Make _csv module thread-safe #118344

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
eeba1c2
Make the filed_limit thread safe
aisk Apr 27, 2024
40e71a8
Make dialects fields on module state thread safe
aisk Apr 27, 2024
c77750f
Utilize FT_ATOMIC_LOAD_INT32
aisk Apr 27, 2024
6fbcb9f
Remove critical section on `dialects` field
aisk Apr 27, 2024
60753f1
Totally revert the change to csv_register_dialect
aisk Apr 29, 2024
64b7d20
Revert field_limit to long, and add atomic api for it
aisk Apr 29, 2024
1de333e
Add _Py_atomic_load_long for other platforms
aisk Apr 29, 2024
7bbfd36
Merge branch 'main' into csv-thread-safe
aisk May 15, 2024
0a41927
Revert the type change to old_limit
aisk May 15, 2024
f413092
Using atomic_store instead of critical_section
aisk May 15, 2024
6c40b59
Update clinic generated files
aisk May 15, 2024
a79cf88
Apply suggestions from code review
aisk May 17, 2024
1f03de1
Merge branch 'main' into csv-thread-safe
aisk Jul 10, 2024
3739c46
Change field_limit to Py_ssize_t
aisk Jul 10, 2024
850bf2a
Remove atomic functions for long type
aisk Jul 10, 2024
bfbfc56
Fix types
aisk Jul 10, 2024
d73b827
Fix the return type of field_size_limit
aisk Jul 14, 2024
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
Prev Previous commit
Next Next commit
Revert the type change to old_limit
  • Loading branch information
@aisk
aisk committed May 15, 2024
commit 0a41927c42d9917afc3717d14d6073858d25a3cf
2 changes: 1 addition & 1 deletion Modules/_csv.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1655,7 +1655,7 @@ _csv_field_size_limit_impl(PyObject *module, PyObject *new_limit)
/*[clinic end generated code: output=f2799ecd908e250b input=3e49d42e37a7d449]*/
{
_csvstate *module_state = get_csv_state(module);
int32_t old_limit = module_state->field_limit;
long old_limit = module_state->field_limit;
if (new_limit != NULL) {
if (!PyLong_CheckExact(new_limit)) {
PyErr_Format(PyExc_TypeError,
Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy