kojiromike commented on a change in pull request #936:
URL: https://github.com/apache/avro/pull/936#discussion_r471003846
##########
File path: lang/py/avro/schema.py
##########
@@ -493,6 +516,17 @@ def __eq__(self, that):
class PrimitiveSchema(Schema):
"""Valid primitive types are in PRIMITIVE_TYPES."""
+ _validators = {
Review comment:
While I started this approach, I realize now that these lambdas don't
show up clearly in test coverage reports. We can leave it this way for now, but
maybe in the future I should move these to named functions.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
