Skip to content
This repository was 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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
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