Avro
  1. Avro
  2. AVRO-215

Add docstring indicating utility of "seen" parameter to __hash__ and __eq__ in schema.py and explore use of decorators for memoization

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: python
    • Labels:
      None

      Description

      As pointed out by Doug on the mailing list, the point of the "seen" parameter is to "contain the set of schemas that are on the stack. Schemas can be circular, so this is required to prevent infinite loops."

        Activity

        Jeff Hammerbacher created issue -
        Jeff Hammerbacher made changes -
        Field Original Value New Value
        Summary Add docstring indicating utility of "seen" parameter to __hash__ and __eq__ in schema.py Add docstring indicating utility of "seen" parameter to __hash__ and __eq__ in schema.py and explore use of decorators for memoization
        Jeff Hammerbacher made changes -
        Assignee Jeff Hammerbacher [ hammer ]
        Jeff Hammerbacher made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development