Avro
  1. Avro
  2. AVRO-428

Improve file read performance by buffering data

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: c
    • Labels:
      None
    1. AVRO-428.patch
      3 kB
      Matt Massie
    2. AVRO-428.patch
      3 kB
      Matt Massie

      Activity

      Matt Massie created issue -
      Hide
      Matt Massie added a comment -

      This patch add buffering for file reads. My benchmarks shows a 2-3x improvement in read speed with this patch.

      Show
      Matt Massie added a comment - This patch add buffering for file reads. My benchmarks shows a 2-3x improvement in read speed with this patch.
      Matt Massie made changes -
      Field Original Value New Value
      Attachment AVRO-428.patch [ 12436638 ]
      Hide
      Doug Cutting added a comment -

      This fails to apply with:

      patch: **** missing line number at line 5: @@ -%ld,%ld +%ld,%ld @@

      Show
      Doug Cutting added a comment - This fails to apply with: patch: **** missing line number at line 5: @@ -%ld,%ld +%ld,%ld @@
      Hide
      Matt Massie added a comment -

      The last patch was incorrect because of a know issue with subversion 1.6.5 and 1.6.6 on MacOS X.

      I'm attaching a new patch generated by git.

      Show
      Matt Massie added a comment - The last patch was incorrect because of a know issue with subversion 1.6.5 and 1.6.6 on MacOS X. I'm attaching a new patch generated by git.
      Matt Massie made changes -
      Attachment AVRO-428.patch [ 12436658 ]
      Hide
      Matt Massie added a comment -

      Committed to trunk

      Show
      Matt Massie added a comment - Committed to trunk
      Matt Massie made changes -
      Status Open [ 1 ] Resolved [ 5 ]
      Resolution Fixed [ 1 ]
      Doug Cutting made changes -
      Fix Version/s 1.3.0 [ 12314318 ]
      Doug Cutting made changes -
      Status Resolved [ 5 ] Closed [ 6 ]
      Transition Time In Source Status Execution Times Last Executer Last Execution Date
      Open Open Resolved Resolved
      20h 4m 1 Matt Massie 23/Feb/10 18:25
      Resolved Resolved Closed Closed
      6d 3h 22m 1 Doug Cutting 01/Mar/10 21:47

        People

        • Assignee:
          Matt Massie
          Reporter:
          Matt Massie
        • Votes:
          0 Vote for this issue
          Watchers:
          1 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development