MINOR: [CI] remove westonpace from CODEOWNERS#50420
Conversation
There was a problem hiding this comment.
Pull request overview
Removes an inactive reviewer from Arrow’s CODEOWNERS entries so C++ acero and engine paths fall back to the existing broader /cpp/src/arrow/ @pitrou ownership.
Changes:
- Deleted explicit CODEOWNERS assignments for
/cpp/src/arrow/aceroand/cpp/src/arrow/engine. - Ensured these subpaths are now covered by the existing
/cpp/src/arrow/ @pitrourule.
kou
left a comment
There was a problem hiding this comment.
+1
BTW, @zanmato1984 are you interested in being a CODEOWNER of them?
I can take the code owner of |
### Rationale for this change Follow-up to #50420. ### What changes are included in this PR? Add an explicit Acero CODEOWNERS entry. ### Are these changes tested? No; CODEOWNERS-only change. ### Are there any user-facing changes? No. Authored-by: Rossi Sun <zanmato1984@gmail.com> Signed-off-by: Rossi Sun <zanmato1984@gmail.com>
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 2f5384f. There were 66 benchmark results with an error:
There were 6 benchmark results indicating a performance regression:
The full Conbench report has more details. It also includes information about 4 possible false positives for unstable benchmarks that are known to sometimes produce them. |
Rationale for this change
I am not actively reviewing PRs for Arrow☹️
What changes are included in this PR?
I removed myself from the CODEOWNERS files in the
aceroandenginesubfolders. Based on the current structure these will fall back to pitrou.Are these changes tested?
No
Are there any user-facing changes?
No