Skip to content

GH-49417: [GLib] Add garrow_map_data_type_is_keys_sorted()#49418

Merged
kou merged 1 commit into
apache:mainfrom
kou:glib-map-data-type-keys-sorted
Mar 2, 2026
Merged

GH-49417: [GLib] Add garrow_map_data_type_is_keys_sorted()#49418
kou merged 1 commit into
apache:mainfrom
kou:glib-map-data-type-keys-sorted

Conversation

@kou

@kou kou commented Mar 1, 2026

Copy link
Copy Markdown
Member

Rationale for this change

It's the bindings of arrow::MapType::keys_sorted().

What changes are included in this PR?

Add garrow_map_data_type_is_keys_sorted().

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown

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

@kou

kou commented Mar 2, 2026

Copy link
Copy Markdown
Member Author

+1

@kou
kou merged commit aae49e8 into apache:main Mar 2, 2026
15 checks passed
@kou kou removed the awaiting committer review Awaiting committer review label Mar 2, 2026
@kou
kou deleted the glib-map-data-type-keys-sorted branch March 2, 2026 01:52
@conbench-apache-arrow

Copy link
Copy Markdown

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

There were no benchmark performance regressions. 🎉

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

thisisnic pushed a commit to thisisnic/arrow that referenced this pull request Apr 6, 2026
…pache#49418)

### Rationale for this change

It's the bindings of `arrow::MapType::keys_sorted()`.

### What changes are included in this PR?

Add `garrow_map_data_type_is_keys_sorted()`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* GitHub Issue: apache#49417

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
…pache#49418)

### Rationale for this change

It's the bindings of `arrow::MapType::keys_sorted()`.

### What changes are included in this PR?

Add `garrow_map_data_type_is_keys_sorted()`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* GitHub Issue: apache#49417

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant