[Bug 65184] New: Improve performance of POFSMiniStore getBlockAt

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 65184] New: Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

            Bug ID: 65184
           Summary: Improve performance of POFSMiniStore getBlockAt
           Product: POI
           Version: 5.0.0-FINAL
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: POIFS
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Follow up to https://bz.apache.org/bugzilla/show_bug.cgi?id=64322

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #1 from PJ Fanning <[hidden email]> ---
committed r1887604

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #2 from PJ Fanning <[hidden email]> ---
the commit is just reformatting - there is still work to do to optimise the
code

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #3 from PJ Fanning <[hidden email]> ---
While the way that the blocks are fetched and the potential for the underlying
to be streamed multiple times is far from ideal - I think the performance
impact will depend a lot on whether the POIFSFileSystem is file backed or byte
array backed. FileBackedDataSource will be affected more.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #4 from PJ Fanning <[hidden email]> ---
added r1887657

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #5 from PJ Fanning <[hidden email]> ---
reverted using r1887660 due to integration test  failures

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65184] Improve performance of POFSMiniStore getBlockAt

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65184

--- Comment #6 from PJ Fanning <[hidden email]> ---
I added back the changes and the integration tests passed

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]