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

Add FilterProject to operator initialize framework #6480

Closed

Conversation

tanjialiang
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Sep 7, 2023

Deploy Preview for meta-velox ready!

Name Link
🔨 Latest commit ab8fbbd
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/65036a14a5970a000853c00d
😎 Deploy Preview https://deploy-preview-6480--meta-velox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@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 Sep 7, 2023
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

Copy link
Contributor

@xiaoxmeng xiaoxmeng left a comment

Choose a reason for hiding this comment

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

@tanjialiang LGTM. Thanks!

@facebook-github-bot
Copy link
Contributor

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

…#6480)

Summary: Pull Request resolved: facebookincubator#6480

Reviewed By: xiaoxmeng

Differential Revision: D49074135

Pulled By: tanjialiang
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@tanjialiang merged this pull request in 79fa20f.

@conbench-facebook
Copy link

Conbench analyzed the 1 benchmark run on commit 79fa20fd.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

codyschierbeck pushed a commit to codyschierbeck/velox that referenced this pull request Sep 27, 2023
…#6480)

Summary: Pull Request resolved: facebookincubator#6480

Reviewed By: xiaoxmeng

Differential Revision: D49074135

Pulled By: tanjialiang

fbshipit-source-id: 8febeb041e9d5c7a50fa7bd4baec458229fed4f6
codyschierbeck pushed a commit to codyschierbeck/velox that referenced this pull request Sep 27, 2023
…#6480)

Summary: Pull Request resolved: facebookincubator#6480

Reviewed By: xiaoxmeng

Differential Revision: D49074135

Pulled By: tanjialiang

fbshipit-source-id: 8febeb041e9d5c7a50fa7bd4baec458229fed4f6
codyschierbeck pushed a commit to codyschierbeck/velox that referenced this pull request Sep 27, 2023
…#6480)

Summary: Pull Request resolved: facebookincubator#6480

Reviewed By: xiaoxmeng

Differential Revision: D49074135

Pulled By: tanjialiang

fbshipit-source-id: 8febeb041e9d5c7a50fa7bd4baec458229fed4f6
ericyuliu pushed a commit to ericyuliu/velox that referenced this pull request Oct 12, 2023
…#6480)

Summary: Pull Request resolved: facebookincubator#6480

Reviewed By: xiaoxmeng

Differential Revision: D49074135

Pulled By: tanjialiang

fbshipit-source-id: 8febeb041e9d5c7a50fa7bd4baec458229fed4f6
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. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants