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.

Sign up for GitHub

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

Jump to bottom

fix(http): geo types should not fall-through to the UUID adapter #3146

Merged
merged 2 commits into from
Apr 4, 2023
Merged

fix(http): geo types should not fall-through to the UUID adapter #3146

merged 2 commits into from
Apr 4, 2023

Conversation

winne42
Copy link
Contributor

@winne42 winne42 commented Apr 1, 2023

I stumbled upon this certainly unintended fall-through behavior in a switch, introduced with #2769

@bluestreak01
Copy link
Member

thank you very much!

@bluestreak01 bluestreak01 changed the title geo types should not fall-through to the UUID adapter fix(sql): geo types should not fall-through to the UUID adapter Apr 3, 2023
@bluestreak01 bluestreak01 changed the title fix(sql): geo types should not fall-through to the UUID adapter fix(http): geo types should not fall-through to the UUID adapter Apr 3, 2023
@bluestreak01 bluestreak01 merged commit 6c98e20 into questdb:master Apr 4, 2023
14 checks passed
@winne42 winne42 deleted the feature/fix_type_fallthrough_for_geo_types branch April 4, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bluestreak01 bluestreak01 bluestreak01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@winne42 @bluestreak01 @ideoma