[
https://issues.apache.org/jira/browse/CRUNCH-293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13825639#comment-13825639
]
Ryan Blue commented on CRUNCH-293:
----------------------------------
One more thing: I was careful to make my branch compatible with the current
method to override the ReflectDataFactory, so that puts off the need to make
simultaneous changes to scrunch. We can also go through a proper deprecation
cycle.
> Injection of reader into AvroRecordReader
> -----------------------------------------
>
> Key: CRUNCH-293
> URL: https://issues.apache.org/jira/browse/CRUNCH-293
> Project: Crunch
> Issue Type: Improvement
> Components: Core
> Affects Versions: 0.7.0, 0.8.0
> Reporter: Micah Whitacre
> Assignee: Micah Whitacre
> Attachments: CRUNCH-293.patch, CRUNCH-293_v2.patch
>
>
> With CRUNCH-243, I wanted to support injecting custom readers to handle the
> cases like passivity between Avro Schema. The changes made however were not
> complete as we also need to be able to inject a reader into the
> AvroRecordReader which constructs its own SpecificDatumReader.
> We could create a SpecificDataFactory which emulates the ReflectDataFactory.
> Or simplify to a single DataFactory which will create either
> Reflect/Specific/Generic. Thoughts?
--
This message was sent by Atlassian JIRA
(v6.1#6144)
