PrimitiveBuilder::finish_dict does not validate dictionary offsets

[tustvold]

Describe the bug

PrimitiveBuilder::finish_dict does not validate that the provided values array contains enough values for the largest key in the dictionary

To Reproduce

#[test]
fn test_primitive_array_builder_dict() {
    let mut builder = Int32Array::builder(10);
    builder.append_value(1).unwrap();

    let values = Arc::new(Int32Array::from_iter_values([0]));
    let dict = builder.finish_dict(values);
}

Expected behavior

It should not be possible to construct invalid ArrayData with safe APIs