Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Plain View
Avro >> mail # dev >> Avro-C: namespace lookup and inherited namespaces


+
Ben Walsh 2013-05-07, 10:17
+
Bruce Mitchener 2013-05-07, 10:30
+
Martin Kleppmann 2013-05-07, 14:03
Copy link to this message
-
Re: Avro-C: namespace lookup and inherited namespaces

It looks like a similar approach -- the parser functions now take the
namespace of the enclosing scope (the Ruby version calls it
"default_namespace" and I call it "parent_namespace"). Plus there's some
extra logic to take into account the fully-qualified name.

Thanks for pointing that out.

Ben

On Tue, 7 May 2013, Martin Kleppmann wrote:

> FWIW, I think the Ruby bug was fixed here:
> https://issues.apache.org/jira/browse/AVRO-1272 � take a look at the patch,
> perhaps you can adopt a similar approach in the C implementation (I don't
> know how similar the code is though, so it may be irrelevant).
>
> Martin
>
>
> On 7 May 2013 02:30, Bruce Mitchener <[EMAIL PROTECTED]> wrote:
>
>> On May 7, 2013, at 5:17 PM, Ben Walsh <[EMAIL PROTECTED]> wrote:
>>
>>>
>>> Hi
>>>
>>> It seems that the C version of Avro doesn't resolve fully-qualified type
>> names, and it doesn't understand inherited namespaces.
>>>
>>> For example this doesn't work:
>>>
>>> {"type": "record", "namespace": "x", "name": "Y", "fields": [
>>>  {"name": "e", "type": {"type": "record", "name": "Z", "fields": [
>>>    {"name": "f", "type": "x.Z"}
>>>  ]}}
>>> ]}
>>>
>>> It can't find the type "x.Z".
>>>
>>> (this is similar to a Ruby problem
>> https://issues.apache.org/jira/browse/AVRO-635).
>>>
>>> I have a patch to make this work. Should I submit a JIRA or is this
>> issue already fixed/not a problem?
>>
>> Sounds like a patch in JIRA would be ideal.
>>
>>  - Bruce
>>
>>
>
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB