Make align_buffers as public API

[viirya]

Which issue does this PR close?

Closes #.

Rationale for this change

Buffer alignment is not automatic for FFI (#4681 (comment)). Although there is ArrayData::align_buffers for that, but it is not public API so we cannot use it outside this crate.

What changes are included in this PR?

Are there any user-facing changes?

[tustvold]

The public API is https://docs.rs/arrow-data/latest/arrow_data/struct.ArrayDataBuilder.html#method.build_aligned

Otherwise you have to build an ArrayData with incorrect alignment to call this method, which is technically unsound per the public API contract

[viirya]

Hmm, but for FFI API, from imported array and schema, you get a ArrayData already instead a builder.

…

On Tue, Sep 26, 2023, 15:28 Raphael Taylor-Davies ***@***.***> wrote: ***@***.**** commented on this pull request. The public API is https://docs.rs/arrow-data/latest/arrow_data/struct.ArrayDataBuilder.html#method.build_aligned Otherwise you have to build an ArrayData with incorrect alignment to call this method, which is technically unsound per the public API contract — Reply to this email directly, view it on GitHub <#4863 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAQZ5ZANDZ6XUX7MCT4IHDX4NJI5ANCNFSM6AAAAAA5IKPF7U> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[tustvold]

Eh, fair enough. Its a bit off, but 😅

We probably should be validating FFI