avro-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Doug Cutting (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AVRO-965) Enhance the IDL parser to allow properties for protocols and messages
Date Mon, 05 Dec 2011 20:32:39 GMT

    [ https://issues.apache.org/jira/browse/AVRO-965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13163020#comment-13163020
] 

Doug Cutting commented on AVRO-965:
-----------------------------------

> Add the private Property class from Schema.java to Protocol.java

I think you mean Props, not Property, is that right?  If so, you might just remove the 'private'
keyword so it becomes package-private.  I don't think we need to make this public in the API.

> Message: NOT WORKING

In idl.jj, can't you simply modify MessageDeclaration to collect properties the same way that
FieldDeclaration does?

Other than that it sounds like you have a good plan.
                
> Enhance the IDL parser to allow properties for protocols and messages
> ---------------------------------------------------------------------
>
>                 Key: AVRO-965
>                 URL: https://issues.apache.org/jira/browse/AVRO-965
>             Project: Avro
>          Issue Type: Improvement
>            Reporter: George Fletcher
>            Priority: Minor
>
> Enhance the IDL parser to support arbitrary properties for protocol and message types.
This will allow for attaching metadata to a protocol or message and can be used for versioning
and in some cases language annotations.
> This was partly discussed as part of JIRA ticket 886
> https://issues.apache.org/jira/browse/AVRO-886

--
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

        

Mime
View raw message