FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Migrate from arrow.Record to arrow.RecordBatch by metalmatze · Pull Request #5966 · parca-dev/parca · GitHub

Migrate from arrow.Record to arrow.RecordBatch - #5966

Merged
metalmatze merged 1 commit into
mainfrom
arrowrecordbatch
Oct 1, 2025
Merged

Migrate from arrow.Record to arrow.RecordBatch#5966
metalmatze merged 1 commit into
mainfrom
arrowrecordbatch

Conversation

Copy link
Copy Markdown
Member

Updated the codebase to replace all usages of arrow.Record with arrow.RecordBatch to make golang-lint happy again.

apache/arrow-go#466 broke our golangci-lint with Arrow v18.4.1

Updated the codebase to replace all usages of `arrow.Record` with `arrow.RecordBatch` to make golang-lint happy again.

alwaysmeticulous Bot commented Oct 1, 2025
edited
Loading

Copy link
Copy Markdown

✅ Meticulous spotted 0 visual differences across 506 screens tested: view results.

Meticulous evaluated ~4 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit 5477607. This comment will update as new commits are pushed.

metalmatze enabled auto-merge (squash) October 1, 2025 15:07
metalmatze merged commit d00dc8d into main Oct 1, 2025
35 checks passed
metalmatze deleted the arrowrecordbatch branch October 1, 2025 15:36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL