Committed by
Gerrit Code Review
Remove methods deprecated in Drake from RoutingConfigurationService API
Change-Id: Ib9f65f8cff9bc0a763b0000fff1996c4404fac39
Showing
2 changed files
with
0 additions
and
45 deletions
... | @@ -74,15 +74,6 @@ public interface RoutingConfigurationService { | ... | @@ -74,15 +74,6 @@ public interface RoutingConfigurationService { |
74 | boolean isIpPrefixLocal(IpPrefix ipPrefix); | 74 | boolean isIpPrefixLocal(IpPrefix ipPrefix); |
75 | 75 | ||
76 | /** | 76 | /** |
77 | - * Retrieves the entire set of interfaces in the network. | ||
78 | - * | ||
79 | - * @return the set of interfaces | ||
80 | - * @deprecated in Drake release - use InterfaceService instead | ||
81 | - */ | ||
82 | - @Deprecated | ||
83 | - Set<Interface> getInterfaces(); | ||
84 | - | ||
85 | - /** | ||
86 | * Retrieves the entire set of connect points connected to BGP peers in the | 77 | * Retrieves the entire set of connect points connected to BGP peers in the |
87 | * network. | 78 | * network. |
88 | * | 79 | * |
... | @@ -91,27 +82,6 @@ public interface RoutingConfigurationService { | ... | @@ -91,27 +82,6 @@ public interface RoutingConfigurationService { |
91 | Set<ConnectPoint> getBgpPeerConnectPoints(); | 82 | Set<ConnectPoint> getBgpPeerConnectPoints(); |
92 | 83 | ||
93 | /** | 84 | /** |
94 | - * Retrieves the interface associated with the given connect point. | ||
95 | - * | ||
96 | - * @param connectPoint the connect point to retrieve interface information | ||
97 | - * for | ||
98 | - * @return the interface | ||
99 | - * @deprecated in Drake release - use InterfaceService instead | ||
100 | - */ | ||
101 | - @Deprecated | ||
102 | - Interface getInterface(ConnectPoint connectPoint); | ||
103 | - | ||
104 | - /** | ||
105 | - * Retrieves the interface associated with the given IP address. | ||
106 | - * | ||
107 | - * @param ip IP address of the interface | ||
108 | - * @return the interface | ||
109 | - * @deprecated in Drake release - use InterfaceService instead | ||
110 | - */ | ||
111 | - @Deprecated | ||
112 | - Interface getInterface(IpAddress ip); | ||
113 | - | ||
114 | - /** | ||
115 | * Retrieves the interface that matches the given IP address. Matching | 85 | * Retrieves the interface that matches the given IP address. Matching |
116 | * means that the IP address is in one of the interface's assigned subnets. | 86 | * means that the IP address is in one of the interface's assigned subnets. |
117 | * | 87 | * | ... | ... |
... | @@ -196,11 +196,6 @@ public class RoutingConfigurationImpl implements RoutingConfigurationService { | ... | @@ -196,11 +196,6 @@ public class RoutingConfigurationImpl implements RoutingConfigurationService { |
196 | } | 196 | } |
197 | 197 | ||
198 | @Override | 198 | @Override |
199 | - public Set<Interface> getInterfaces() { | ||
200 | - return Collections.emptySet(); | ||
201 | - } | ||
202 | - | ||
203 | - @Override | ||
204 | public Set<ConnectPoint> getBgpPeerConnectPoints() { | 199 | public Set<ConnectPoint> getBgpPeerConnectPoints() { |
205 | // TODO perhaps cache this result in future | 200 | // TODO perhaps cache this result in future |
206 | ApplicationId routerAppId = coreService.getAppId(Router.ROUTER_APP_ID); | 201 | ApplicationId routerAppId = coreService.getAppId(Router.ROUTER_APP_ID); |
... | @@ -222,16 +217,6 @@ public class RoutingConfigurationImpl implements RoutingConfigurationService { | ... | @@ -222,16 +217,6 @@ public class RoutingConfigurationImpl implements RoutingConfigurationService { |
222 | } | 217 | } |
223 | 218 | ||
224 | @Override | 219 | @Override |
225 | - public Interface getInterface(ConnectPoint connectPoint) { | ||
226 | - return null; | ||
227 | - } | ||
228 | - | ||
229 | - @Override | ||
230 | - public Interface getInterface(IpAddress ip) { | ||
231 | - return null; | ||
232 | - } | ||
233 | - | ||
234 | - @Override | ||
235 | public Interface getMatchingInterface(IpAddress ipAddress) { | 220 | public Interface getMatchingInterface(IpAddress ipAddress) { |
236 | return null; | 221 | return null; |
237 | } | 222 | } | ... | ... |
-
Please register or login to post a comment