Thomas Vachuska | 4f1a60c | 2014-10-28 13:39:07 -0700 | [diff] [blame] | 1 | /* |
Ray Milkey | 34c9590 | 2015-04-15 09:47:53 -0700 | [diff] [blame^] | 2 | * Copyright 2014-2015 Open Networking Laboratory |
Thomas Vachuska | 4f1a60c | 2014-10-28 13:39:07 -0700 | [diff] [blame] | 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 16 | package org.onosproject.net.topology.impl; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 17 | |
Abhishek Dwaraki | 1e5873e | 2015-03-08 00:01:17 -0500 | [diff] [blame] | 18 | import static com.google.common.base.Strings.isNullOrEmpty; |
| 19 | import static java.util.concurrent.Executors.newFixedThreadPool; |
Thomas Vachuska | 6519e6f | 2015-03-11 02:29:31 -0700 | [diff] [blame] | 20 | import static org.onlab.util.Tools.get; |
Abhishek Dwaraki | 1e5873e | 2015-03-08 00:01:17 -0500 | [diff] [blame] | 21 | import static org.onlab.util.Tools.groupedThreads; |
| 22 | import static org.onosproject.core.CoreService.CORE_PROVIDER_ID; |
| 23 | import static org.onosproject.net.device.DeviceEvent.Type.DEVICE_ADDED; |
| 24 | import static org.onosproject.net.device.DeviceEvent.Type.DEVICE_AVAILABILITY_CHANGED; |
| 25 | import static org.onosproject.net.device.DeviceEvent.Type.DEVICE_REMOVED; |
| 26 | import static org.slf4j.LoggerFactory.getLogger; |
| 27 | |
| 28 | import java.util.Collections; |
| 29 | import java.util.Dictionary; |
| 30 | import java.util.List; |
| 31 | import java.util.Timer; |
| 32 | import java.util.concurrent.ExecutorService; |
| 33 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 34 | import org.apache.felix.scr.annotations.Activate; |
| 35 | import org.apache.felix.scr.annotations.Component; |
| 36 | import org.apache.felix.scr.annotations.Deactivate; |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 37 | import org.apache.felix.scr.annotations.Modified; |
| 38 | import org.apache.felix.scr.annotations.Property; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 39 | import org.apache.felix.scr.annotations.Reference; |
| 40 | import org.apache.felix.scr.annotations.ReferenceCardinality; |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 41 | import org.apache.felix.scr.annotations.Service; |
Thomas Vachuska | ecb63c5 | 2015-02-19 10:03:48 -0800 | [diff] [blame] | 42 | import org.onlab.util.AbstractAccumulator; |
| 43 | import org.onlab.util.Accumulator; |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 44 | import org.onosproject.cfg.ComponentConfigService; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 45 | import org.onosproject.event.Event; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 46 | import org.onosproject.net.device.DeviceEvent; |
| 47 | import org.onosproject.net.device.DeviceListener; |
| 48 | import org.onosproject.net.device.DeviceService; |
| 49 | import org.onosproject.net.link.LinkEvent; |
| 50 | import org.onosproject.net.link.LinkListener; |
| 51 | import org.onosproject.net.link.LinkService; |
| 52 | import org.onosproject.net.provider.AbstractProvider; |
| 53 | import org.onosproject.net.topology.DefaultGraphDescription; |
| 54 | import org.onosproject.net.topology.GraphDescription; |
| 55 | import org.onosproject.net.topology.TopologyProvider; |
| 56 | import org.onosproject.net.topology.TopologyProviderRegistry; |
| 57 | import org.onosproject.net.topology.TopologyProviderService; |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 58 | import org.osgi.service.component.ComponentContext; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 59 | import org.slf4j.Logger; |
| 60 | |
Abhishek Dwaraki | 1e5873e | 2015-03-08 00:01:17 -0500 | [diff] [blame] | 61 | import com.google.common.collect.ImmutableList; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 62 | |
| 63 | /** |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 64 | * Default implementation of a network topology provider that feeds off |
| 65 | * device and link subsystem events to trigger assembly and computation of |
| 66 | * new topology snapshots. |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 67 | */ |
| 68 | @Component(immediate = true) |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 69 | @Service |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 70 | public class DefaultTopologyProvider extends AbstractProvider |
| 71 | implements TopologyProvider { |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 72 | |
Thomas Vachuska | a132e3a | 2015-02-21 01:53:14 -0800 | [diff] [blame] | 73 | private static final int MAX_THREADS = 8; |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 74 | private static final int DEFAULT_MAX_EVENTS = 1000; |
| 75 | private static final int DEFAULT_MAX_IDLE_MS = 10; |
| 76 | private static final int DEFAULT_MAX_BATCH_MS = 50; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 77 | |
tom | 025e09f | 2014-09-15 15:29:24 -0700 | [diff] [blame] | 78 | // FIXME: Replace with a system-wide timer instance; |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 79 | // TODO: Convert to use HashedWheelTimer or produce a variant of that; then decide which we want to adopt |
Thomas Vachuska | a132e3a | 2015-02-21 01:53:14 -0800 | [diff] [blame] | 80 | private static final Timer TIMER = new Timer("onos-topo-event-batching"); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 81 | |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 82 | @Property(name = "maxEvents", intValue = DEFAULT_MAX_EVENTS, |
| 83 | label = "Maximum number of events to accumulate") |
| 84 | private int maxEvents = DEFAULT_MAX_EVENTS; |
| 85 | |
| 86 | @Property(name = "maxIdleMs", intValue = DEFAULT_MAX_IDLE_MS, |
| 87 | label = "Maximum number of millis between events") |
| 88 | private int maxIdleMs = DEFAULT_MAX_IDLE_MS; |
| 89 | |
| 90 | @Property(name = "maxBatchMs", intValue = DEFAULT_MAX_BATCH_MS, |
| 91 | label = "Maximum number of millis for whole batch") |
| 92 | private int maxBatchMs = DEFAULT_MAX_BATCH_MS; |
| 93 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 94 | private final Logger log = getLogger(getClass()); |
| 95 | |
| 96 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 97 | protected TopologyProviderRegistry providerRegistry; |
| 98 | |
| 99 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 100 | protected DeviceService deviceService; |
| 101 | |
| 102 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 103 | protected LinkService linkService; |
| 104 | |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 105 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 106 | protected ComponentConfigService cfgService; |
| 107 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 108 | private volatile boolean isStarted = false; |
| 109 | |
| 110 | private TopologyProviderService providerService; |
Abhishek Dwaraki | 1e5873e | 2015-03-08 00:01:17 -0500 | [diff] [blame] | 111 | private final DeviceListener deviceListener = new InternalDeviceListener(); |
| 112 | private final LinkListener linkListener = new InternalLinkListener(); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 113 | |
Thomas Vachuska | ecb63c5 | 2015-02-19 10:03:48 -0800 | [diff] [blame] | 114 | private Accumulator<Event> accumulator; |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 115 | private ExecutorService executor; |
| 116 | |
| 117 | /** |
| 118 | * Creates a provider with the supplier identifier. |
| 119 | */ |
tom | 9793755 | 2014-09-11 10:48:42 -0700 | [diff] [blame] | 120 | public DefaultTopologyProvider() { |
Thomas Vachuska | 6acd3bb | 2014-11-09 23:44:22 -0800 | [diff] [blame] | 121 | super(CORE_PROVIDER_ID); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 122 | } |
| 123 | |
| 124 | @Activate |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 125 | public synchronized void activate(ComponentContext context) { |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 126 | cfgService.registerProperties(DefaultTopologyProvider.class); |
| 127 | executor = newFixedThreadPool(MAX_THREADS, groupedThreads("onos/topo", "build-%d")); |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 128 | accumulator = new TopologyChangeAccumulator(); |
| 129 | logConfig("Configured"); |
| 130 | |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 131 | modified(context); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 132 | |
| 133 | providerService = providerRegistry.register(this); |
| 134 | deviceService.addListener(deviceListener); |
| 135 | linkService.addListener(linkListener); |
| 136 | |
| 137 | isStarted = true; |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 138 | triggerRecompute(); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 139 | log.info("Started"); |
| 140 | } |
| 141 | |
| 142 | @Deactivate |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 143 | public synchronized void deactivate(ComponentContext context) { |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 144 | cfgService.unregisterProperties(DefaultTopologyProvider.class, false); |
tom | e52ce70 | 2014-09-11 00:12:54 -0700 | [diff] [blame] | 145 | isStarted = false; |
| 146 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 147 | deviceService.removeListener(deviceListener); |
| 148 | linkService.removeListener(linkListener); |
| 149 | providerRegistry.unregister(this); |
| 150 | providerService = null; |
| 151 | |
| 152 | executor.shutdownNow(); |
| 153 | executor = null; |
| 154 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 155 | log.info("Stopped"); |
| 156 | } |
| 157 | |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 158 | @Modified |
| 159 | public void modified(ComponentContext context) { |
| 160 | if (context == null) { |
| 161 | accumulator = new TopologyChangeAccumulator(); |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 162 | logConfig("Reconfigured"); |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 163 | return; |
| 164 | } |
| 165 | |
Thomas Vachuska | 6519e6f | 2015-03-11 02:29:31 -0700 | [diff] [blame] | 166 | Dictionary<?, ?> properties = context.getProperties(); |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 167 | int newMaxEvents, newMaxBatchMs, newMaxIdleMs; |
| 168 | try { |
Thomas Vachuska | 6519e6f | 2015-03-11 02:29:31 -0700 | [diff] [blame] | 169 | String s = get(properties, "maxEvents"); |
Ayaka Koshibe | 8851ed9 | 2015-01-22 12:07:24 -0800 | [diff] [blame] | 170 | newMaxEvents = isNullOrEmpty(s) ? maxEvents : Integer.parseInt(s.trim()); |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 171 | |
Thomas Vachuska | 6519e6f | 2015-03-11 02:29:31 -0700 | [diff] [blame] | 172 | s = get(properties, "maxBatchMs"); |
Ayaka Koshibe | 8851ed9 | 2015-01-22 12:07:24 -0800 | [diff] [blame] | 173 | newMaxBatchMs = isNullOrEmpty(s) ? maxBatchMs : Integer.parseInt(s.trim()); |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 174 | |
Thomas Vachuska | 6519e6f | 2015-03-11 02:29:31 -0700 | [diff] [blame] | 175 | s = get(properties, "maxIdleMs"); |
Ayaka Koshibe | 8851ed9 | 2015-01-22 12:07:24 -0800 | [diff] [blame] | 176 | newMaxIdleMs = isNullOrEmpty(s) ? maxIdleMs : Integer.parseInt(s.trim()); |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 177 | |
Pavlin Radoslavov | b9e50df | 2015-02-20 20:01:26 -0800 | [diff] [blame] | 178 | } catch (NumberFormatException | ClassCastException e) { |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 179 | newMaxEvents = DEFAULT_MAX_EVENTS; |
| 180 | newMaxBatchMs = DEFAULT_MAX_BATCH_MS; |
| 181 | newMaxIdleMs = DEFAULT_MAX_IDLE_MS; |
| 182 | } |
| 183 | |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 184 | if (newMaxEvents != maxEvents || newMaxBatchMs != maxBatchMs || newMaxIdleMs != maxIdleMs) { |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 185 | maxEvents = newMaxEvents; |
| 186 | maxBatchMs = newMaxBatchMs; |
| 187 | maxIdleMs = newMaxIdleMs; |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 188 | accumulator = maxEvents > 1 ? new TopologyChangeAccumulator() : null; |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 189 | logConfig("Reconfigured"); |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 190 | } |
| 191 | } |
| 192 | |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 193 | private void logConfig(String prefix) { |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 194 | log.info("{} with maxEvents = {}; maxBatchMs = {}; maxIdleMs = {}; accumulator={}", |
| 195 | prefix, maxEvents, maxBatchMs, maxIdleMs, accumulator != null); |
Thomas Vachuska | 6b7920d | 2014-11-25 19:48:39 -0800 | [diff] [blame] | 196 | } |
| 197 | |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 198 | |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 199 | @Override |
| 200 | public void triggerRecompute() { |
| 201 | triggerTopologyBuild(Collections.<Event>emptyList()); |
| 202 | } |
| 203 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 204 | /** |
| 205 | * Triggers assembly of topology data citing the specified events as the |
| 206 | * reason. |
| 207 | * |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 208 | * @param reasons events which triggered the topology change |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 209 | */ |
tom | e52ce70 | 2014-09-11 00:12:54 -0700 | [diff] [blame] | 210 | private synchronized void triggerTopologyBuild(List<Event> reasons) { |
tom | 9793755 | 2014-09-11 10:48:42 -0700 | [diff] [blame] | 211 | if (executor != null) { |
| 212 | executor.execute(new TopologyBuilderTask(reasons)); |
| 213 | } |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 214 | } |
| 215 | |
| 216 | // Builds the topology using the latest device and link information |
| 217 | // and citing the specified events as reasons for the change. |
| 218 | private void buildTopology(List<Event> reasons) { |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 219 | if (isStarted) { |
tom | 9793755 | 2014-09-11 10:48:42 -0700 | [diff] [blame] | 220 | GraphDescription desc = |
| 221 | new DefaultGraphDescription(System.nanoTime(), |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 222 | System.currentTimeMillis(), |
| 223 | deviceService.getAvailableDevices(), |
| 224 | linkService.getActiveLinks()); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 225 | providerService.topologyChanged(desc, reasons); |
| 226 | } |
| 227 | } |
| 228 | |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 229 | private void processEvent(Event event) { |
| 230 | if (accumulator != null) { |
| 231 | accumulator.add(event); |
| 232 | } else { |
| 233 | triggerTopologyBuild(ImmutableList.of(event)); |
| 234 | } |
| 235 | } |
| 236 | |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 237 | // Callback for device events |
Ayaka Koshibe | 3de43ca | 2014-09-26 16:40:23 -0700 | [diff] [blame] | 238 | private class InternalDeviceListener implements DeviceListener { |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 239 | @Override |
| 240 | public void event(DeviceEvent event) { |
| 241 | DeviceEvent.Type type = event.type(); |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 242 | if (type == DEVICE_ADDED || type == DEVICE_REMOVED || |
| 243 | type == DEVICE_AVAILABILITY_CHANGED) { |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 244 | processEvent(event); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 245 | } |
| 246 | } |
| 247 | } |
| 248 | |
| 249 | // Callback for link events |
Ayaka Koshibe | 3de43ca | 2014-09-26 16:40:23 -0700 | [diff] [blame] | 250 | private class InternalLinkListener implements LinkListener { |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 251 | @Override |
| 252 | public void event(LinkEvent event) { |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 253 | processEvent(event); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 254 | } |
| 255 | } |
| 256 | |
| 257 | // Event accumulator for paced triggering of topology assembly. |
Thomas Vachuska | ecb63c5 | 2015-02-19 10:03:48 -0800 | [diff] [blame] | 258 | private class TopologyChangeAccumulator extends AbstractAccumulator<Event> { |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 259 | TopologyChangeAccumulator() { |
Thomas Vachuska | 912bdd5 | 2014-11-17 11:39:01 -0800 | [diff] [blame] | 260 | super(TIMER, maxEvents, maxBatchMs, maxIdleMs); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 261 | } |
| 262 | |
| 263 | @Override |
Thomas Vachuska | ecb63c5 | 2015-02-19 10:03:48 -0800 | [diff] [blame] | 264 | public void processItems(List<Event> items) { |
| 265 | triggerTopologyBuild(items); |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 266 | } |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 267 | } |
| 268 | |
| 269 | // Task for building topology data in a separate thread. |
| 270 | private class TopologyBuilderTask implements Runnable { |
| 271 | private final List<Event> reasons; |
| 272 | |
| 273 | public TopologyBuilderTask(List<Event> reasons) { |
| 274 | this.reasons = reasons; |
| 275 | } |
| 276 | |
| 277 | @Override |
| 278 | public void run() { |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 279 | try { |
| 280 | buildTopology(reasons); |
| 281 | } catch (Exception e) { |
Thomas Vachuska | 6d697f1 | 2015-03-08 20:59:50 -0700 | [diff] [blame] | 282 | log.warn("Unable to compute topology due to: {}", e.getMessage()); |
Yuta HIGUCHI | 2210282 | 2014-11-12 23:09:59 -0800 | [diff] [blame] | 283 | log.debug("Unable to compute topology", e); |
Thomas Vachuska | 0e752bd | 2014-10-22 22:33:41 -0700 | [diff] [blame] | 284 | } |
tom | cbff939 | 2014-09-10 00:45:23 -0700 | [diff] [blame] | 285 | } |
| 286 | } |
| 287 | |
| 288 | } |