-
Notifications
You must be signed in to change notification settings - Fork 133
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
[EJBCLIENT-175] Add getUri() method to Affinity hierarchy #171
Conversation
Build 204 outcome was FAILURE using a merge of d21432c Failed tests
|
d21432c
to
fb454a1
Compare
Build 205 outcome was FAILURE using a merge of fb454a1 Failed tests
|
fb454a1
to
294e8f0
Compare
Apparently I've gone 15 years without knowing that java.net.URI won't accept "xxx:" as a valid URI. |
|
||
static { | ||
try { | ||
uri = new URI("local", "-", null); |
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'm not super happy about this. But Java doesn't support conforming "empty" URIs with only a scheme part. Still thinking of a way to sling this weakness into an advantage, somehow.
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 still don't like this; trying to think of a better way, but if anyone has a suggestion or idea please speak up.
294e8f0
to
c10df57
Compare
No description provided.