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

refactor(RowNumberFuzzer): Cleanup redundant code and includes #12282

Closed
wants to merge 1 commit into from

Conversation

aditi-pandit
Copy link
Collaborator

No description provided.

@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 Feb 7, 2025
Copy link

netlify bot commented Feb 7, 2025

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit e3e18df
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/67a7a424ffebd20008406fac

@aditi-pandit aditi-pandit force-pushed the cleanup_row_number_fuzzer branch from 7ebf116 to 08ed87b Compare February 7, 2025 03:04
@aditi-pandit aditi-pandit force-pushed the cleanup_row_number_fuzzer branch from 08ed87b to fa28e46 Compare February 7, 2025 17:17
Copy link
Contributor

@kagamiori kagamiori left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@aditi-pandit aditi-pandit force-pushed the cleanup_row_number_fuzzer branch from fa28e46 to e3e18df Compare February 8, 2025 18:36
@aditi-pandit aditi-pandit 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 Feb 8, 2025
@aditi-pandit
Copy link
Collaborator Author

@kagamiori has approved above.

@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 merged this pull request in 05cf8f0.

@aditi-pandit aditi-pandit deleted the cleanup_row_number_fuzzer branch February 12, 2025 07:25
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.

3 participants