On 8/9/11 5:53 PM, "Bill Graham" <[email protected]> wrote:
> I see that Doug's already got an IDL patch in place for AVRO-872, so I'll take > a look at that (tonight hopefully) and comment on it. Maybe these two items > are one in the same though... > > Scott I like your approach, but I wonder if it could be simplified. Maybe we > add two new methods like this instead: > > public Schema Schema.parse(File[] files); // (1) > public Schema Schema.parse(File[] files, Map<Name, Schema> context); // (2) > > 1 - This allows the caller to not have to make multiple calls and handle state > if they don't want to. > 2 - This does the same, but also exposes the context, which would be handy to > easily grab a specific schema. Now that I say it I wonder if (1) would ever be > useful on it's own without having the context? I agree, the API can be made much simpler for common use cases. At this point, the right place for continued conversation and ideas is AVRO-872 https://issues.apache.org/jira/browse/AVRO-872
