Skip to content
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.

Sign up for GitHub

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

Jump to bottom

loadbalance LocalFirstLoadBalance #42

Closed
yxlHuster opened this issue May 11, 2016 · 1 comment
Closed

loadbalance LocalFirstLoadBalance #42

yxlHuster opened this issue May 11, 2016 · 1 comment
Labels
bug

Comments

@yxlHuster
Copy link

  1. LocalFirstLoadBalance.searchLocalReferer
    long local = ipToLong(localhost);
    放在循环里面是不是有点浪费？
  2. LocalFirstLoadBalance.doSelectToHolder
    if (localReferers.isEmpty()) {
    Collections.sort(localReferers, new LowActivePriorityComparator());
    refersHolder.addAll(localReferers);
    }
    也不太对劲
@qdaxb
Copy link
Collaborator

qdaxb commented May 11, 2016

已修改 #46

@qdaxb qdaxb added the bug label May 11, 2016
half-dead added a commit that referenced this issue May 12, 2016
@half-dead
Merge pull request #46 from qdaxb/fix_localfirst_bug
e1e2e3a 
fix bug of localfirst loadbalance, fix #42
qdaxb added a commit to qdaxb/motan that referenced this issue May 13, 2016
@qdaxb
fix bug of localfirst loadbalance, fix weibocom#42
8e48637
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@qdaxb @yxlHuster