Avro
  1. Avro
  2. AVRO-284

Handle namespaces correctly in new Python implementation

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0
    • Component/s: python
    • Labels:
      None

      Description

      Once AVRO-219 goes in, we should add unit tests and ensure that we handle namespaces as specified in AVRO-253.

      1. AVRO-284.patch.txt
        16 kB
        Patrick Wendell
      2. AVRO-284-v1.patch
        17 kB
        Jeff Hammerbacher
      3. AVRO-284-v2.patch
        6 kB
        Patrick Wendell

        Issue Links

          Activity

          Hide
          Patrick Wendell added a comment -

          This patch updates python implementation to include appropriate namespace logic. In particular, uses an implicit default namespace when full names are unspecified. Includes unit-tests.

          Show
          Patrick Wendell added a comment - This patch updates python implementation to include appropriate namespace logic. In particular, uses an implicit default namespace when full names are unspecified. Includes unit-tests.
          Hide
          Jeff Hammerbacher added a comment -

          Awesome work, Patrick! I cleaned up some of the comment styling and added a few extra tests, but this is ready to commit. Thanks again.

          Show
          Jeff Hammerbacher added a comment - Awesome work, Patrick! I cleaned up some of the comment styling and added a few extra tests, but this is ready to commit. Thanks again.
          Hide
          Jeff Hammerbacher added a comment -

          Committed to trunk with revision 953190.

          Show
          Jeff Hammerbacher added a comment - Committed to trunk with revision 953190.
          Hide
          Jeff Hammerbacher added a comment -

          Turns out this patch breaks schema printing. We need to maintain state about how a schema is created (e.g. which of name/namespace is set) and definitely not synthesize a property called "fullname". My apologies for the poor review, and for the poor test coverage which did not catch it.

          Show
          Jeff Hammerbacher added a comment - Turns out this patch breaks schema printing. We need to maintain state about how a schema is created (e.g. which of name/namespace is set) and definitely not synthesize a property called "fullname". My apologies for the poor review, and for the poor test coverage which did not catch it.
          Hide
          Jeff Hammerbacher added a comment -

          Todd set us up with a code review tool so we can pick through the exact issues. See http://review.hbase.org/r/166/ for the most recent version of the patch. Let's continue the discussion there.

          Show
          Jeff Hammerbacher added a comment - Todd set us up with a code review tool so we can pick through the exact issues. See http://review.hbase.org/r/166/ for the most recent version of the patch. Let's continue the discussion there.
          Hide
          Patrick Wendell added a comment -

          Hey Jeff think this addresses everything we talked about.

          Show
          Patrick Wendell added a comment - Hey Jeff think this addresses everything we talked about.
          Hide
          Jeff Hammerbacher added a comment -

          Committed revision 955129 with the additional fixes.

          Show
          Jeff Hammerbacher added a comment - Committed revision 955129 with the additional fixes.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development