You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@servicecomb.apache.org by GitBox <gi...@apache.org> on 2022/09/06 04:01:24 UTC

[GitHub] [servicecomb-java-chassis] etteeee commented on pull request #3223: [SCB-2643]fix instance isolation filter not work problem

etteeee commented on PR #3223:
URL: https://github.com/apache/servicecomb-java-chassis/pull/3223#issuecomment-1237632484

   Now we have changed isolation and zoneaware type from `DiscoveryFilters` to `ServerListFilterEx`, so what is the real difference between this 2 types filter except for the filtering timing? What is the **RULE** when adding a new filter policy? @liubao68 @zhaozw 
   
   btw, it's better to create a new class with new name rather than moving to a new package, the class name may cause unnecessary confusion.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@servicecomb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org