Skip to content

Which parts of this library are threadsafe? #464

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.

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

Closed
laian opened this issue Sep 27, 2018 · 6 comments
Closed

Which parts of this library are threadsafe? #464

laian opened this issue Sep 27, 2018 · 6 comments

Comments

@laian
Copy link

laian commented Sep 27, 2018

Is this library in general threadsafe?

Assuming that I preload all schemas for the RefResolver:

Can I share validators across threads?

If not, can I have 1 validator per thread, but have all validators across threads share the same RefResolver?

@Julian
Copy link
Member

Julian commented Sep 29, 2018

Hey -- in general no objects are explicitly guaranteed to be threadsafe (and no tests currently deal with threads).

If all you're trying to do is share some large dicts, adding them to a bunch of different RefResolver stores doesn't copy them.

@Julian
Copy link
Member

Julian commented Sep 29, 2018

#223 might also interest you.

@laian
Copy link
Author

laian commented Oct 1, 2018

thanks!

@laian laian closed this as completed Oct 1, 2018
@stkrp
Copy link

stkrp commented Oct 23, 2019

Hello.

Sorry to bring up the old closed topic, but is it safe to pass the same object (dict) to different validators (one validator per thread)?

Example

I doubt this code works, but I hope that's enough for an example.

import threading
from multiprocessing.pool import ThreadPool

from jsonschema import Draft7Validator


heavy_schema = {
    # Includes several `$ref` like "#/definitions/some_struct"
    # ...
}


class ThreadSafeValidator:
    def __init__(self, schema):
        self._schema = schema
        self._tl = threading.local()
    
    def validate(self, value):
        if not hasattr(self._tl, "validator"):
            # Create independent `Draft7Validator` for each thread.
            # Can I use `_schema` without deepcopy?
            self._tl.validator = Draft7Validator(_schema)
        
        self._tl.validator.validate(value)


validator = ThreadSafeValidator(heavy_schema)


def validate_value(value):
    validator.validate(value)


pool = ThreadPool(4)
pool.map(validate_value, range(1000))
pool.close()
pool.join()

@Julian
Copy link
Member

Julian commented Oct 23, 2019 via email

@stkrp
Copy link

stkrp commented Oct 24, 2019

Great! Thanks.

Julian added a commit that referenced this issue Apr 13, 2021
15ec577 Merge pull request #471 from json-schema-org/ether/id-anchor-in-enum
9f97865 test for confusing not-identifiers in enums
0f7ecd4 Merge pull request #475 from marksparkza/marksparkza-patch-1
783d22a Add jschon
fc68499 Merge pull request #472 from json-schema-org/ether/unevaluatedProperties_uncles
ed4cf5f more test cases for unevaluatedItems, unevaluatedProperties
d0d814d Merge pull request #469 from json-schema-org/ether/ipv4-vulnerability
7ca5f36 reject ipv4 strings with an octet with a leading zero
8e1e1c1 fix spelling error in test descriptions
77f1d10 Merge pull request #462 from jdesrosiers/dynamic-ref-tests
72a32fe Merge pull request #468 from json-schema-org/ether/combine-test-cases
0c48ffb Merge pull request #453 from notEthan/float-overflow-d4-int
76a4ba0 these test cases can be combined since the schemas are the same
cd73775 Merge pull request #464 from json-schema-org/ether/format-by-default-always-validates
043dc63 by default, "format" only annotates, not validates
3c45b81 Merge pull request #460 from amosonn/remove-remotes-from-script
b09e48d Fix $ref with siblings in pre-2019-09 tests
ff9f22e Add tests for $dynamicRef/$dynamicAnchor
0faaf09 Fix refs to Draft 2019-09 schema to be refs to 2020-12
ebbcbc8 Use flask to server remotes directly
bb98b03 Remove remotes from bin/jsonschema_suite
fcae732 Merge pull request #455 from jdesrosiers/bootstrap-202012
e002409 Update tests for 2020-12
405b3fb Copy 2019-09 tests to bootstrap 2020-12 tests
1636a22 draft4 float-overflow instance may be considered not an integer
8daea3f Merge pull request #451 from json-schema-org/ether/more-relative-json-pointer
69fe40f some more relative-json-pointer tests
6505944 Merge pull request #450 from json-schema-org/ether/recursiveRef-dynamic-path
afd0cd3 Move content* keyword tests to non-optional
e2b2a4b Change all content* keyword tests to always validate
8999eae $recursiveRef example demonstrating dynamic nature of the resolution scope
f47003f fix duplicate test description
bcf1dc8 Merge pull request #391 from ether/recursiveRef (rebased, squashed)
3d88f34 test $recursiveRef + $recursiveAnchor
3b79a45 Merge pull request #418 from ChALkeR/chalker/contentSchema
29f609b Add tests for contentSchema

git-subtree-dir: json
git-subtree-split: 15ec577f5ddee0115319f4e7f856cd57567a9c78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants