[ARROW-9550] [Rust] [DataFusion] Remove Rc<RefCell<_>> from hash aggregate operator

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Fix Version/s: 2.0.0
• Component/s: Rust, Rust - DataFusion
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/25616
• Assignee: Unassigned
• Reporter: Andy Grove
• Created: 24/Jul/20 13:22
• Updated: 11/Jan/23 08:07
• Resolved: 19/Aug/20 13:26

Description

The hasg aggregate operator runs on a thread and therefore does not require Rc<RefCell<_>> usage and could just use mutable references instead.

This does require some refactoring of the logic so that the code simply iterates over the rows, calling get_mut on the hashmap per row.

Issue Links

• links to: GitHub Pull Request #7848