Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix velox_dwio_parquet_reader_benchmark build on Ubuntu #10648

Closed

Conversation

majetideepak
Copy link
Collaborator

Resolves #10323

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 1, 2024
Copy link

netlify bot commented Aug 1, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit a911731
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/66ac1a3bbed91700089a0511

@majetideepak majetideepak changed the title Fix velox_dwio_parquet_reader_benchmark on Ubuntu Fix velox_dwio_parquet_reader_benchmark build on Ubuntu Aug 1, 2024
@majetideepak majetideepak requested a review from assignUser August 1, 2024 23:29
@majetideepak majetideepak added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Aug 2, 2024
@facebook-github-bot
Copy link
Contributor

@kagamiori has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

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

I confirmed that this fixes benchmark builds for me. Thanks!

@facebook-github-bot
Copy link
Contributor

@kagamiori merged this pull request in 07bfc61.

Copy link

Conbench analyzed the 1 benchmark run on commit 07bfc616.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

facebook-github-bot pushed a commit that referenced this pull request Aug 6, 2024
Summary:
For some reason the linux debug build has started to fail because it get's oom killed during linking: https://github.com/facebookincubator/velox/actions/runs/10259255187/job/28383420958#step:7:6891

I am not sure if the addition of parquet in #10648 is to blame, in that PR the job did not fail.

Pull Request resolved: #10671

Reviewed By: kevinwilfong

Differential Revision: D60841583

Pulled By: kagamiori

fbshipit-source-id: f3d4a679baf2739f8ff316b535377036527be07d
@majetideepak majetideepak deleted the fix-benchmark branch October 5, 2024 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ubuntu build failing when VELOX_ENABLE_PARQUET flag is enabled
4 participants