GH-45649: [GLib] Add GArrowBinaryViewArray

[hiroyuki-sato]

Rationale for this change

arrow::BinaryViewArray is available in the C++ API.
But, GLib doesn't support that method yet.

What changes are included in this PR?

Add GArrowBinaryViewArray for wrapping arrow::BinaryViewArray class

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

• GitHub Issue: [GLib] Add GArrowBinaryViewArray #45649

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - Contributing Overview

[github-actions]

⚠️ GitHub issue #45649 has been automatically assigned in GitHub to PR creator.

[kou]

Can we improve variable name something like the following?

Suggested change

	buffer_string = "test"
	short_binary_data = "test"

[kou]

Can we improve variable name something like the following?

Suggested change

	view_bytes = 16
	short_string_space = 12

(I think that we don't need to include length space here.)

FYI: The specification: https://arrow.apache.org/docs/format/Columnar.html#variable-size-binary-view-layout

[hiroyuki-sato]

I got the exception Error: test_new(TestBinaryViewArray): Arrow::Error::Invalid: [array][validate-full]: Invalid: Buffer #1 too small in array of type binary_view and length 1: expected at least 16 byte(s), got 8 if I don't use padding.

  def test_new
    # https://arrow.apache.org/docs/format/Columnar.html#variable-size-binary-view-layout
    short_binary_data = "test"
    short_view_buffer_space = 12
    short_view_buffer = [short_binary_data.size].pack("l")
    #short_view_buffer += short_binary_data.ljust(short_view_buffer_space, "\x00")
    short_view_buffer += short_binary_data

    arrow_view_buffer = Arrow::Buffer.new(short_view_buffer)
    arrow_data_buffer = Arrow::Buffer.new(short_binary_data)
    bitmap = Arrow::Buffer.new([0b1].pack("C*"))

    binary_view_array = Arrow::BinaryViewArray.new(1,
                                                   arrow_view_buffer,
                                                   [arrow_data_buffer],
                                                   bitmap,
                                                   0,
                                                   0)
    assert do
      binary_view_array.validate_full
    end
    assert_equal(short_binary_data, binary_view_array.get_value(0).to_s)
  end

[kou]

+1

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit d4d26fe.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 11 possible false positives for unstable benchmarks that are known to sometimes produce them.