Skip to content

Draft reader/codec factory draft - WIP.#51

Draft
cmnbroad wants to merge 1 commit intomasterfrom
cn_codec_discovery
Draft

Draft reader/codec factory draft - WIP.#51
cmnbroad wants to merge 1 commit intomasterfrom
cn_codec_discovery

Conversation

@cmnbroad
Copy link
Copy Markdown
Collaborator

@cmnbroad cmnbroad commented Mar 4, 2019

This isn't ready for line-by-line review; more just for discussion at the next meeting.

Draft implementation of a dynamic HtsjdkCodec discovery and registration service, implemented via a java.util.ServiceLoader. Each HtsjdkCodec consists of two classes; a lightweight, dynamically discovered singleton HtsjdkCodecDescriptor, which is cached in a registry, and the actual HtsjdkCodec itself, which is only instantiated with actual inputs when the corresponding HtsjdkCodecDescriptor claims to be able to handle a given input resource.
This PR contains a single stub/mock BAMCodec implementation as a test case for exercising the discovery/registry code.

This does not include any changes discussed during the last meeting(s). In particular:

  • we still need to define an additional service layer for use by codecs to map/resolve input resources to actual consumable channel/stream/resources
  • a way to provide sibling file indentifiers directly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant