The current ChunkInputStream implementation: a) incorrectly implements the skip() operation, potentially causing corrupt data to be read b) when multiple chunks are present, uses a sequential algorithm for moving across chunks during the skip operation. These 2 issues should be fixed.
Horia Chiorean <hchiorea> made a comment on jira MODE-1752 The information that allowed "direct" streaming was already present in the Metadata and the chunk streams: the size of a chunk, the number of chunks and the total size of the binary. Therefore, the change is 100% backwards compatible. This PR contains 2 commits: - one which fixes and implements the direct skipping mechanism - one which exposes the "chunkSize" as a JSON and AS7 configuration property, allowing clients to "fine-tune" their ISPN binary storage if needed.
Horia Chiorean <hchiorea> made a comment on jira MODE-1752 The information that allowed "direct" skipping was already present in the Metadata and the chunk streams: the size of a chunk, the number of chunks and the total size of the binary. Therefore, the change is 100% backwards compatible. This PR contains 2 commits: - one which fixes and implements the direct skipping mechanism - one which exposes the "chunkSize" as a JSON and AS7 configuration property, allowing clients to "fine-tune" their ISPN binary storage if needed.
Randall Hauch <rhauch> made a comment on jira MODE-1752 Rebased and merged into the 'master' branch. Leaving this issue open until it can be merged into the 3.3.x-prod-ip6.0 branch.
This issue contains 2 commits: 1) https://github.com/jboss-integration/modeshape/commit/36a99be85153ab825580f263cec13b35195e1bfc 2) https://github.com/jboss-integration/modeshape/commit/fd71be4e8185f3d4af4892f0911dd39e86c4acf3
Randall Hauch <rhauch> updated the status of jira MODE-1752 to Closed
Fixed before GA of major release - setting to requires_doc_text- accordingly