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
Remove set data type #1010
Remove set data type #1010
Conversation
Also in the code the generates schemas.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked at this thoroughly, have run the tests locally as well. Just some minor remarks, mostly LGTM.
@@ -1 +1 @@ | |||
:addr in { 192.168.1.1, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.1, 192.168.1.255, 192.168.1.1, 192.168.1.1, 192.168.1.1, 224.0.0.251, ff02::fb, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.1, 192.168.1.255, 224.0.0.251} | |||
:addr in [ 192.168.1.1, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.1, 192.168.1.255, 192.168.1.1, 192.168.1.1, 192.168.1.1, 224.0.0.251, ff02::fb, 192.168.1.255, 192.168.1.255, 192.168.1.255, 192.168.1.1, 192.168.1.255, 224.0.0.251] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make sense for this query to still have duplicated fields?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The old set
never enforced uniqueness, so now that this is a vector
, it's the exact same behavior.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But now that you mention it, we probably should have a story to perform a "uniquing" pass when using in
with vector
operands.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes. Can you take care of filing a ticket for the optimization for in
operations with lists that you suggested?
No description provided.