[
https://issues.apache.org/jira/browse/AVRO-3256?focusedWorklogId=683943&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-683943
]
ASF GitHub Bot logged work on AVRO-3256:
----------------------------------------
Author: ASF GitHub Bot
Created on: 19/Nov/21 13:54
Start Date: 19/Nov/21 13:54
Worklog Time Spent: 10m
Work Description: opwvhk commented on a change in pull request #1407:
URL: https://github.com/apache/avro/pull/1407#discussion_r753208508
##########
File path:
lang/java/compiler/src/main/javacc/org/apache/avro/compiler/idl/idl.jj
##########
@@ -1141,7 +1141,7 @@ List<String> EnumBody():
}
{
"{"
- [ EnumConstant(symbols) ( LOOKAHEAD(2) "," EnumConstant(symbols) )* ]
Review comment:
This lookahead was never needed.
##########
File path:
lang/java/compiler/src/main/javacc/org/apache/avro/compiler/idl/idl.jj
##########
@@ -1141,7 +1141,7 @@ List<String> EnumBody():
}
{
"{"
- [ EnumConstant(symbols) ( LOOKAHEAD(2) "," EnumConstant(symbols) )* ]
Review comment:
This lookahead was not needed.
--
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: 683943)
Time Spent: 20m (was: 10m)
> 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: 20m
> 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)
