[GitHub] [pulsar] hangc0276 commented on pull request #12232: [Python] Do not sort schema fields by default

[GitBox <gi...@apache.org>]

hangc0276 commented on pull request #12232:
URL: https://github.com/apache/pulsar/pull/12232#issuecomment-929866753


   @merlimat Some tests run failed, Please take a look, thanks.
   ```
   FAIL: test_avro_required_default (schema_test.SchemaTest)
   ----------------------------------------------------------------------
   Traceback (most recent call last):
     File "/tmp/schema_test.py", line 856, in test_avro_required_default
       "default": None
   AssertionError: {'fields': [{'default': None, 'type': ['null', 'int'], 'name': 'a'}, {'default': [truncated]... != {'fields': [{'default': None, 'type': ['null', 'int'], 'name': 'a'}, {'default': [truncated]...
   Diff is 2313 characters long. Set self.maxDiff to None to see it.
   
   ======================================================================
   FAIL: test_complex (schema_test.SchemaTest)
   ----------------------------------------------------------------------
   Traceback (most recent call last):
     File "/tmp/schema_test.py", line 106, in test_complex
       "type": ["null", 'MySubRecord']
   AssertionError: {'fields': [{'type': ['null', 'string'], 'name': 'a'}, {'type': ['null', {'field [truncated]... != {'fields': [{'type': ['null', 'string'], 'name': 'a'}, {'type': ['null', {'field [truncated]...
   Diff is 1195 characters long. Set self.maxDiff to None to see it.
   ```


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org