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 IPPREFIX to/from VARCHAR casting #11309

Closed

Conversation

mohsaka
Copy link
Contributor

@mohsaka mohsaka commented Oct 21, 2024

Add IPPREFIX to/from VARCHAR casting.

Fuzzer changes here:
#11209

@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 21, 2024
Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit b9bdeab
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/672d5e30eda088000814e0dc

@mohsaka mohsaka closed this Oct 28, 2024
@mohsaka mohsaka force-pushed the ipprefix_cast_varchar branch from 6c84027 to 878388f Compare October 28, 2024 20:40
@mohsaka mohsaka reopened this Oct 28, 2024
@mohsaka mohsaka force-pushed the ipprefix_cast_varchar branch 5 times, most recently from fd28300 to de55f21 Compare October 29, 2024 17:00
@mohsaka mohsaka force-pushed the ipprefix_cast_varchar branch 2 times, most recently from f6bbdce to 53b1d30 Compare November 6, 2024 19:08
@mohsaka mohsaka requested a review from aditi-pandit November 6, 2024 23:36
facebook-github-bot pushed a commit that referenced this pull request Nov 7, 2024
Summary:
When switching from varbinary based to row based ipprefix the typeof was not changed. Changed here to fix the issue.

Split from:
#11309

Pull Request resolved: #11449

Reviewed By: yuandagits

Differential Revision: D65612144

Pulled By: kevinwilfong

fbshipit-source-id: 8fc716f0c8e78a1355727b2c1179c9955828c16c
@mohsaka mohsaka force-pushed the ipprefix_cast_varchar branch from 53b1d30 to 2ee8c07 Compare November 8, 2024 00:27
@mohsaka mohsaka requested a review from czentgr November 8, 2024 00:30
@mohsaka mohsaka force-pushed the ipprefix_cast_varchar branch from 2ee8c07 to b9bdeab Compare November 8, 2024 00:41
@mohsaka
Copy link
Contributor Author

mohsaka commented Dec 4, 2024

Merged in via
#11591

@mohsaka mohsaka closed this Dec 4, 2024
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.

4 participants