mattjackson220 commented on a change in pull request #4857:
URL: https://github.com/apache/trafficcontrol/pull/4857#discussion_r464673611
##########
File path:
traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/util/AbstractResourceWatcher.java
##########
@@ -57,7 +57,7 @@ public void configure(final JsonNode config) {
}
final WatcherConfig watcherConfig = new
WatcherConfig(getWatcherConfigPrefix(), config, trafficOpsUtils);
- final String resourceUrl = (watcherConfig.getUrl() != null &&
!watcherConfig.getUrl().isEmpty()) ? watcherConfig.getUrl() : getDataBaseURL();
+ final String resourceUrl = (watcherConfig.getUrl() != null &&
!watcherConfig.getUrl().isEmpty()) ? watcherConfig.getUrl() :
getDefaultDatabaseURL();
Review comment:
i only see it used below in a conditional to check if it has been
changed and then it gets set at the bottom to the new value if needed. am i
missing other uses?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
