[
https://issues.apache.org/jira/browse/AVRO-3256?focusedWorklogId=684650&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-684650
]
ASF GitHub Bot logged work on AVRO-3256:
----------------------------------------
Author: ASF GitHub Bot
Created on: 22/Nov/21 12:05
Start Date: 22/Nov/21 12:05
Worklog Time Spent: 10m
Work Description: martin-g commented on a change in pull request #1407:
URL: https://github.com/apache/avro/pull/1407#discussion_r754204005
##########
File path: doc/src/content/xdocs/idl.xml
##########
@@ -429,50 +429,62 @@ record MyRecord {
<p>Some annotations like those listed above are handled
specially. All other annotations are added as properties to
the protocol, message, schema or field.</p>
+ <p>Note that for named types, annotations should be added to
+ the type definition; they cannot be added to the type references.</p>
</section>
</section>
<section id="example">
<title>Complete Example</title>
- <p>The following is a complete example of a Avro IDL file that shows
most of the above features:</p>
+ <p>The following is an example of an Avro IDL file that shows most of
the above features:</p>
<source>
+/*
+* Header with license information.
+*/
+
/**
* An example protocol in Avro IDL
*/
@namespace("org.apache.avro.test")
protocol Simple {
-
+ /** Documentation for the enum type Kind */
Review comment:
Not directly related to this PR but I've noticed that using comments
starting with `/**` leads to `doc` values starting with `* `.
So, one should either use `/* ... */` or the parser should be improved to
support both syntaxes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 684650)
Time Spent: 1h 10m (was: 1h)
> IDL: annotations on type references change the referenced type
> --------------------------------------------------------------
>
> Key: AVRO-3256
> URL: https://issues.apache.org/jira/browse/AVRO-3256
> Project: Apache Avro
> Issue Type: Bug
> Components: java, tools
> Affects Versions: 1.11.0
> Reporter: Oscar Westra van Holthe - Kind
> Assignee: Oscar Westra van Holthe - Kind
> Priority: Major
> Labels: javacc, pull-request-available
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> The IDL compiler allows annotations on type references, and applies them on
> the referenced type.
> As can be seen in the test output file {{simple.avpr}}, the type {{MD5}} has
> the property {{"foo"="bar"}}.
> But in the input file {{simple.avdl}}:
> * The type definition in lines 39-40 has no such property
> * Line 53 adds the property to the type for the field {{hash}},
> * But it's also silently added to the type for the field {{nullableHash}} in
> line 55
> Solution: do not accept annotations for type references.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
