[jira] [Updated] (AVRO-1169) Reduce memory footprint of ResolvingGrammarGenerator

["Doug Cutting (JIRA)" <ji...@apache.org>]

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

Doug Cutting updated AVRO-1169:
-------------------------------

    Attachment: AVRO-1169.patch

Here is the proposed change as a patch file.
                
> Reduce memory footprint of ResolvingGrammarGenerator
> ----------------------------------------------------
>
>                 Key: AVRO-1169
>                 URL: https://issues.apache.org/jira/browse/AVRO-1169
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.7.0
>            Reporter: Hernan Otero
>         Attachments: AVRO-1169.patch
>
>
> We noticed an OutOfMemory error in one of our client applications (GUI) when trying to decode messages that require resolving.
> It turns out the use of Schema.toString(true) in method generate(Schema, Schema, Map<LitS,Symbol>) (and to a less extent resolveRecords(Schema, Schema, Map<LitS,Symbol>) too) in our very-large-Schema use-case is the main reason for this.
> We found that patching this class to use Schema.getFullName() instead of Schema.toString(true) resolves our issue, and we thought this might benefit other users (and shouldn't affect anybody as this message is not actually being logged/reported).
> The change involves changing this (in method generate()):
> {code}
>     return Symbol.error("Found " + writer.toString(true)
>                         + ", expecting " + reader.toString(true));
> {code}
> into this:
> {code}
>     return Symbol.error("Found " + writer.getFullName()
>                         + ", expecting " + reader.getFullName());
> {code}
> And this (in method resolveRecords()):
> {code}
>             result = Symbol.error("Found " + writer.toString(true)
>                                   + ", expecting " + reader.toString(true));
> {code}
> into this:
> {code}
>             result = Symbol.error("Found " + writer.getFullName()
>                                   + ", expecting " + reader.getFullName());
> {code}
> Thanks in advance,
> Hernan

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira