[
https://issues.apache.org/jira/browse/BEAM-3324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16290224#comment-16290224
]
ASF GitHub Bot commented on BEAM-3324:
--------------------------------------
herohde opened a new pull request #4256: [BEAM-3324] Cache Go runtime symbol
lookups
URL: https://github.com/apache/beam/pull/4256
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> symtab.go shouldn't read entire file into memory
> ------------------------------------------------
>
> Key: BEAM-3324
> URL: https://issues.apache.org/jira/browse/BEAM-3324
> Project: Beam
> Issue Type: Improvement
> Components: sdk-go
> Reporter: Bill Neubauer
> Assignee: Bill Neubauer
> Priority: Minor
>
> The implementation of symtab.go reads the entire binary into memory. This is
> wasteful of memory, and it should just use os.File as the backing reader. If
> performance becomes an issue, we can use a modest amount of memory to cache
> lookups and avoid filesystem reads.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
