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 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development