Move `RunArray::get_physical_indices` to `RunEndBuffer`

Original context: https://github.com/apache/arrow-rs/pull/9019/changes#r2635583677

**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**


The `RunArray::get_physical_indices` method technically operates solely on the `RunEndBuffer` of the array:

https://github.com/apache/arrow-rs/blob/addf74d72d88684dabac21822c22f35e803135c5/arrow-array/src/array/run_array.rs#L197-L204

**Describe the solution you'd like**


Move this implementation to [`RunEndBuffer`](https://github.com/apache/arrow-rs/blob/addf74d72d88684dabac21822c22f35e803135c5/arrow-buffer/src/buffer/run.rs), and leave `RunArray::get_physical_indices` as a thin wrapper. Maybe also deprecate it?

**Describe alternatives you've considered**


**Additional context**


No sure how it affects inlining, since we do have the `#[inline]` attribute present 🤔

	// TODO: this technically should be a method on RunEndBuffer
	#[inline]
	pub fn get_physical_indices<I>(&self, logical_indices: &[I]) -> Result<Vec<usize>, ArrowError>
	where
	I: ArrowNativeType,
	{
	let len = self.run_ends().len();
	let offset = self.run_ends().offset();

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Move `RunArray::get_physical_indices` to `RunEndBuffer` #9025

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Move RunArray::get_physical_indices to RunEndBuffer #9025

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

Move `RunArray::get_physical_indices` to `RunEndBuffer` #9025