[prev in list] [next in list] [prev in thread] [next in thread] 

List:       avro-dev
Subject:    [jira] [Updated] (AVRO-1026) Add namespace support to C++
From:       "Keh-Li Sheng (Updated) (JIRA)" <jira () apache ! org>
Date:       2012-02-23 22:33:49
Message-ID: 618248584.12218.1330036429202.JavaMail.tomcat () hel ! zones ! apache ! org
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/AVRO-1026?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Keh-Li Sheng updated AVRO-1026:
-------------------------------

    Fix Version/s: 1.6.3
           Status: Patch Available  (was: Open)

Adding submit patch tag for more visibility. Previously commented on by @thiru_mg but \
nothing has happened since.  
> Add namespace support to C++
> ----------------------------
> 
> Key: AVRO-1026
> URL: https://issues.apache.org/jira/browse/AVRO-1026
> Project: Avro
> Issue Type: Improvement
> Components: c++
> Affects Versions: 1.6.1
> Reporter: Keh-Li Sheng
> Labels: patch
> Fix For: 1.6.3
> 
> Attachments: AVRO-1026.patch, AVRO-1026_json.patch, AVRO-1026_with_AVRO-956.patch
> 
> 
> This patch adds support for reading and resolving schemas that use namespaces for \
> named types. Looking for a code review and application of the patch to codebase. \
> Node is modified to take an additional NamespaceConcept attribute in the template \
> parameters. An additional stack has been added to the compiler context to track \
> when namespaces have been entered or left for resolving further types/symbols. \
> Symbolic resolution is now done against the "fullname" of the type and not just by \
> the name. Some string gymnastics were needed in other areas of codegen in order to \
> handle the new symbols. Added very trivial tests to AvrogencppTests for schema \
> generation by avrogencpp and added a test schema (tweet) that has namespaces, a \
> record definition inside a namespace, and then a later symbolic reference by name \
> within an outer namespace. Also patched to work with gen-cppcode.py output. Note \
> that NodeImpl::printBasicInfo intentionally does not output the namespace since \
> that caused downstream breakage of gen-cppcode.py expected format. github project \
> was forked at https://github.com/bifflabs/avro

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: \
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more \
information on JIRA, see: http://www.atlassian.com/software/jira

        


[prev in list] [next in list] [prev in thread] [next in thread]