Uploaded image for project: 'Avro'
  1. Avro
  2. AVRO-293

Add tests for default field values in io.py

    Details

    • Type: Test
    • Status: Closed
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: python
    • Labels:
      None
    1. AVRO-293.patch
      11 kB
      Jeff Hammerbacher

      Activity

      Hide
      hammer Jeff Hammerbacher added a comment -

      Some trickiness:

      • Simplejson parses a JSON null as a Python None
      • Need to sort out encoding issues for bytes
      Show
      hammer Jeff Hammerbacher added a comment - Some trickiness: Simplejson parses a JSON null as a Python None Need to sort out encoding issues for bytes
      Hide
      hammer Jeff Hammerbacher added a comment -

      Had to make some changes to schema parsing to accommodate default values for null schemas, and had to update the spec to indicate the limitation on unicode code points allowed as code points for default values for bytes schemas, but default values work for all schema types now.

      Show
      hammer Jeff Hammerbacher added a comment - Had to make some changes to schema parsing to accommodate default values for null schemas, and had to update the spec to indicate the limitation on unicode code points allowed as code points for default values for bytes schemas, but default values work for all schema types now.
      Hide
      hammer Jeff Hammerbacher added a comment -

      Patch has gone stale; will fold work into AVRO-289

      Show
      hammer Jeff Hammerbacher added a comment - Patch has gone stale; will fold work into AVRO-289
      Hide
      hammer Jeff Hammerbacher added a comment -

      Will be handled in AVRO-289

      Show
      hammer Jeff Hammerbacher added a comment - Will be handled in AVRO-289

        People

        • Assignee:
          hammer Jeff Hammerbacher
          Reporter:
          hammer Jeff Hammerbacher
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development