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
Feature: Add IPv6 Match Function #15212
Feature: Add IPv6 Match Function #15212
Conversation
processing/src/main/java/org/apache/druid/query/expression/IPv6AddressMatchExprMacro.java
Fixed
Show resolved
Hide resolved
Hi - is there an approx ETA for a release containing this change? |
It will depend when this PR gets reviewed. I've been keeping an eye on it but unfortunately I don't have any way of asking maintainers to have a look. Doesn't look like this will get in prior to the 28.0 milestone but hopefully it will be in the one afterwards (reviews dependent!) |
@LakshSingla, @abhishekagarwal87 Sorry to bother you both but would it be possible to get a PR review on this please? Just been open a while and I would welcome your feedback! |
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.
overall lgtm
processing/src/main/java/org/apache/druid/query/expression/IPv6AddressMatchExprMacro.java
Outdated
Show resolved
Hide resolved
...java/org/apache/druid/sql/calcite/expression/builtin/IPv6AddressMatchOperatorConversion.java
Outdated
Show resolved
Hide resolved
Made suggested changes, ready for another review :) |
Fixes #10817.
Description
Added a SQL function to match IPv6 addresses to a given subnet. This was based off the existing
IPv4_MATCH
function. I initially considered changing the existingIPv4_MATCH
function to be a more generic (e.g.IP_MATCH
). However, given this has been a function for some time I thought it best to keep the existing function for existing queries and make an IPv6 specific implementation. Like the existing IPv4 implementation, this relies on the Javainet.ipaddr
libraryRelease note
Added
IPv6_MATCH
SQL function for matching IPv6 addresses in a subnetKey added classes in this PR
IPv6AddressExprUtils
IPv6AddressMatchExprMacro
This PR has: