-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-24777 InfoServer support ipv6 host and port #2147
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
Conversation
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
@Apache9 @infraio @anoopsjohn mind have a look ,small patch ,don't know why yetus failed and the patch doesn't apply branch-2.2/branch-2.1, what should I do ? make another push , like : push origin origin branch-2.1:HBASE-214777 |
virajjasani
left a comment
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.
+1, nice change. Can you also add unit test for IPv6 host, port?
branch-2.1 is dead, so just ignore it. Please open another PR for branch-2.2. |
|
And for the UT, I think it could be another issue. I do not know if we have any IPv6 related UTs in our code base, as it has special requirements on the env? |
|
Even if we have some way to mock, it would be fine, but anyways, can be taken as a separate task. |
virajjasani
left a comment
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.
+1
|
thanks for review @virajjasani @Apache9 |
|
#2154 for branch-2.2 |
Closes #2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]>
Closes #2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]>
Closes #2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]>
Closes #2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]>
Closes apache#2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]>
Closes apache#2147 Signed-off-by: Duo Zhang <[email protected]> Signed-off-by: Viraj Jasani <[email protected]> (cherry picked from commit 5eb4b87) Change-Id: I355d602a00192e607bca7c23beadbf6c82da059c
HBASE-24777 InfoServer support ipv6 host and port