Details
-
Improvement
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
Description
Why we do this?
- When implement invocation based filter, e.g. darklaunch deployment, we need to filter servers based on invocation parameters. But now, we create a Loadbalaner instance for each service, and Robin filter interface can not pass Invocation to it. Before refactor, we doing this by ThreadLocal and cache Invocation, and use in filter implementation. This is error prone and users can't handle this complixity
- We have other improvements, e.g.
SCB-700, we can do server status check, and add a filter a choose from active servers. We need to consider filter orders and other feature, current implementation not satisfy this. - We want to make loadbalance code handy by remove some compatible fixes.