Skip to content

GH-50139: [Dev][Gandiva] Add Gandiva code owners#50144

Merged
kou merged 2 commits into
apache:mainfrom
kou:dev-code-owners-gandiva
Jun 14, 2026
Merged

GH-50139: [Dev][Gandiva] Add Gandiva code owners#50144
kou merged 2 commits into
apache:mainfrom
kou:dev-code-owners-gandiva

Conversation

@kou

@kou kou commented Jun 10, 2026

Copy link
Copy Markdown
Member

Rationale for this change

We have Gandiva PRs that wait for review.

If we have Gandiva code owners, they can notice them easily.

What changes are included in this PR?

Add Gandiva code owners.

Are these changes tested?

No.

Are there any user-facing changes?

No.

@kou

kou commented Jun 10, 2026

Copy link
Copy Markdown
Member Author

@dmitry-chirkov-dremio @lriggs @akravchukdremio @xxlaykxx Could you check this?

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Jun 10, 2026
@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jun 12, 2026
@kou kou merged commit 316d4ef into apache:main Jun 14, 2026
9 checks passed
@kou kou deleted the dev-code-owners-gandiva branch June 14, 2026 07:40
@kou kou removed the awaiting merge Awaiting merge label Jun 14, 2026
@conbench-apache-arrow

Copy link
Copy Markdown

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants