Avro
  1. Avro
  2. AVRO-907

optimize access to protobuf messages

    Details

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

      Description

      AVRO-804 included an optimization to Thrift struct field access that's also applicable to protobuf.

      1. AVRO-907.patch
        7 kB
        Doug Cutting

        Activity

        Doug Cutting created issue -
        Hide
        Doug Cutting added a comment -

        Here's a patch that implements this same optimization for protobuf. Instead of accessing the cache that maps Avro field positions to protobuf field ids once per field the cache is only accessed once per record when reading or writing.

        Show
        Doug Cutting added a comment - Here's a patch that implements this same optimization for protobuf. Instead of accessing the cache that maps Avro field positions to protobuf field ids once per field the cache is only accessed once per record when reading or writing.
        Doug Cutting made changes -
        Field Original Value New Value
        Attachment AVRO-907.patch [ 12497721 ]
        Hide
        Doug Cutting added a comment -

        I'll commit this soon.

        Show
        Doug Cutting added a comment - I'll commit this soon.
        Doug Cutting made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Doug Cutting added a comment -

        I committed this.

        Show
        Doug Cutting added a comment - I committed this.
        Doug Cutting made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2m 45s 1 Doug Cutting 04/Oct/11 23:49
        Patch Available Patch Available Resolved Resolved
        19h 48m 1 Doug Cutting 05/Oct/11 19:38
        Resolved Resolved Closed Closed
        26d 23h 57m 1 Doug Cutting 01/Nov/11 18:36

          People

          • Assignee:
            Doug Cutting
            Reporter:
            Doug Cutting
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development