[ARROW-3337] [JS] IPC writer doesn't serialize the dictionary of nested Vectors

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: JS-0.3.1
• Fix Version/s: JS-0.4.0
• Component/s: JavaScript
• External issue URL: https://github.com/apache/arrow/issues/19670
• Assignee: Paul Taylor
• Reporter: Paul Taylor
• Created: 27/Sep/18 00:11
• Updated: 11/Jan/23 07:26
• Resolved: 13/Jan/19 20:14

Description

The JS writer only serializes dictionaries for top-level children of a Table. This is wrong, and an oversight on my part. The fix here is to put the actual Dictionary vectors in the `schema.dictionaries` map instead of the dictionary fields, like I understand the C++ does.

Issue Links

• is related to: ARROW-3523 [JS] Assign dictionary IDs in IPC writer rather than on creation