Add DictionaryArray::with_values to make it easier to operate on dictionary values #2798
Conversation
| /// Returns a new dictionary with the same keys as the current instance | ||
| /// but with a different set of dictionary values | ||
| /// | ||
| /// This can be used to perform an operation on the values of a dictionary |
| /// ``` | ||
| /// | ||
| pub fn with_values(&self, values: &dyn Array) -> Self { | ||
| assert!(values.len() >= self.values.len()); |
There was a problem hiding this comment.
It took me a little while to convince myself it would be ok if values had a different datatype than self.values()? Because the dictionary builder will create the correct DataType::Dictionary for the new array
alamb
changed the title
|
Benchmark runs are scheduled for baseline = 0052d25 and contender = 41dd12d. 41dd12d is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
Closes #2797
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?