[BEAM-3324] symtab.go shouldn't read entire file into memory

• Type: Improvement
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: sdk-go
• Assignee: Bill Neubauer
• Reporter: Bill Neubauer
• Due: 15/Dec/17
• Created: 11/Dec/17 20:53
• Updated: 16/May/20 13:19
• Resolved: 26/Feb/18 21:36

Description

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.

Issue Links

• links to: GitHub Pull Request #4256
• links to: GitHub Pull Request #4269