Thanks a lot for the useful comments Ryan. 1) https://issues.apache.org/jira/browse/AVRO-695 (Cyclical references) is just ready to be patched in. It has all the suggestions from Doug Cutting and an exhaustive testcase. Doug Cutting, if you can respond to this JIRA ticket, your help would be much appreciated.
2) https://issues.apache.org/jira/browse/AVRO-680 (Non-string map keys) is also completely ready for review. Please go ahead for the review. 3) https://issues.apache.org/jira/browse/AVRO-1554 (Supporting UUID in Avro) Yes, I will remove the date-part and re-submit the patch. Thanks Sachin On Mon, Feb 2, 2015 at 3:36 PM, Ryan Blue <[email protected]> wrote: > On 01/26/2015 11:03 AM, S G wrote: > >> Hi, >> >> I have submitted a couple of patches that need some review. >> 1) https://issues.apache.org/jira/browse/AVRO-695 (Cyclical references) >> > > This looks complicated and has a long history, so I'll try to help out on > the other two. > > 2) https://issues.apache.org/jira/browse/AVRO-680 (Non-string map keys) >> > > Looks like the latest on this is some discussion about whether or not it > would be useful. In my opinion, it would be a good addition. Other formats > and processing engines don't have the restriction that keys are strings. > For example, Parquet allows any object to be a key and Hive allows any > primitive type. > > Is there anything else blocking this, or is the current patch ready for > review? > > 3) https://issues.apache.org/jira/browse/AVRO-1554 (Supporting UUID in >> Avro) >> > > This notes some updates that need to be done as of September last year > (remove Date), but the last patch is from June. Do you want to update the > patch? > > rb > > > -- > Ryan Blue > Software Engineer > Cloudera, Inc. >
