Skip to content

GH-50456: [Ruby] Add ArrowFormat::DurationType#==#50460

Merged
kou merged 1 commit into
apache:mainfrom
Reranko05:gh-50456-duration-type-eq
Jul 10, 2026
Merged

GH-50456: [Ruby] Add ArrowFormat::DurationType#==#50460
kou merged 1 commit into
apache:mainfrom
Reranko05:gh-50456-duration-type-eq

Conversation

@Reranko05

@Reranko05 Reranko05 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Rationale for this change

Comparing ArrowFormat::DurationType instances with the same unit should return true.

What changes are included in this PR?

Add ArrowFormat::DurationType#==.

Are these changes tested?

Yes.

Added tests for:

  • equal duration types
  • different duration types

Are there any user-facing changes?

Yes.

GitHub Issue: #50456

Signed-off-by: Aaditya Srinivasan <aadityasri03@gmail.com>
@Reranko05 Reranko05 requested a review from kou as a code owner July 10, 2026 05:24
Copilot AI review requested due to automatic review settings July 10, 2026 05:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

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

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit 0e7c4ea into apache:main Jul 10, 2026
34 checks passed
@kou kou removed the awaiting review Awaiting review label Jul 10, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Jul 10, 2026
@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 0e7c4ea.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

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.

3 participants