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
[JAV-585]Loadbalance rule dynamic configuration is not work at some time #454
Conversation
boolean isRuleNotChanged = StringUtils.equals(policy,this.policy) && StringUtils.equals(strategy, this.strategy); | ||
|
||
if (!isRuleNotChanged){ | ||
boolean isRuleNotChanged = isEqual(policy, this.policy) && isEqual(strategy,this.strategy); |
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.
not format code?
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.
Yeah, I forget it and I have modified
Assert.assertEquals(false, globalNotNull); | ||
boolean localNotNull = handler.isEqual("com.netflix.loadbalancer.RandomRule", null); | ||
Assert.assertEquals(false, localNotNull); | ||
|
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.
delete empty lines
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.
okay, and I finished
@laijianbin I don't think you logged a right JIRA ID. Can you double check the JIRA ID? |
@WillemJiang create a task at opensource JIRA and use opensource JIRA ID? |
Changes Unknown when pulling 662d8c9 on laijianbin:jhfre into ** on apache:master**. |
Follow this checklist to help us incorporate your contribution quickly and easily:
[JAV-XXX] Fixes bug in ApproximateQuantiles
, where you replaceJAV-XXX
with the appropriate JIRA issue.mvn clean install
to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.