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

Create PyVelox package and move type dependencies over. #503

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

Conversation

kgpai
Copy link
Contributor

@kgpai kgpai commented Oct 4, 2022

PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

@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 Oct 4, 2022
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

kgpai added a commit that referenced this pull request Oct 13, 2022
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: 83427b5aee17eef5470f93129200fcd215286fa2
@facebook-github-bot
Copy link
Contributor

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

kgpai added a commit that referenced this pull request Oct 14, 2022
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: f1404f4e1284bd8f0f39e547348d228c3fe11abd
@facebook-github-bot
Copy link
Contributor

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

kgpai added a commit that referenced this pull request Oct 14, 2022
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Reviewed By: wenleix

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: 36a6a14e9c00b881a1d5c40512ac9fa36a2199a2
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

kgpai added a commit that referenced this pull request Oct 14, 2022
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Reviewed By: wenleix

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: 93ce2c3892343e4ef9bbe01589b51ef20efa15eb
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Reviewed By: wenleix

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: 069aed8ecf2275b0cc0ce3e223fe7d83e0678589
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit that referenced this pull request Oct 17, 2022
Summary:
PyVelox package is gated with CMake flags in this pr. This refactoring will allow Torcharrow and Koski to use a common set of bindings.

Pull Request resolved: #503

Reviewed By: wenleix

Differential Revision: D40082189

Pulled By: kgpai

fbshipit-source-id: 925f3808cc8429e36d37e084abf73df2596a6346
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants