Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

Decouple the building of benchmarks and tests (#2439) #491

Open
wants to merge 1 commit into
base: fbsync
Choose a base branch
from

Conversation

kgpai
Copy link
Contributor

@kgpai kgpai commented Sep 14, 2022

Summary:
Tests and benchmarks targets are now de-coupled.
That means they can be built independently.
Shared functionality is moved to a common utility library.

Resolves facebookincubator/velox#1704

X-link: facebookincubator/velox#2439

Reviewed By: Yuhta

Differential Revision: D39484543

Pulled By: kgpai

Summary:
Tests and benchmarks targets are now de-coupled.
That means they can be built independently.
Shared functionality is moved to a common utility library.

Resolves facebookincubator/velox#1704

X-link: facebookincubator/velox#2439

Reviewed By: Yuhta

Differential Revision: D39484543

Pulled By: kgpai

fbshipit-source-id: b8b0f3a2fec4651ee8b4fe9ca6304bd37b9be183
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Sep 14, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D39484543

facebook-github-bot pushed a commit to facebookincubator/velox that referenced this pull request Sep 15, 2022
Summary:
X-link: pytorch/torcharrow#491

Tests and benchmarks targets are now de-coupled.
That means they can be built independently.
Shared functionality is moved to a common utility library.

Resolves #1704

Pull Request resolved: #2439

Reviewed By: Yuhta

Differential Revision: D39484543

Pulled By: kgpai

fbshipit-source-id: 5ac888c81a6bbfbc5a1a1c4cfd41fa2c86199bc4
facebook-github-bot pushed a commit that referenced this pull request Sep 15, 2022
Summary:
Pull Request resolved: #491

Tests and benchmarks targets are now de-coupled.
That means they can be built independently.
Shared functionality is moved to a common utility library.

Resolves facebookincubator/velox#1704

X-link: facebookincubator/velox#2439

Reviewed By: Yuhta

Differential Revision: D39484543

Pulled By: kgpai

fbshipit-source-id: 5ac888c81a6bbfbc5a1a1c4cfd41fa2c86199bc4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants