- 11 Feb, 2016 3 commits
-
-
Dave Syer authored
-
Dave Syer authored
-
Spencer Gibb authored
fixes gh-822
-
- 09 Feb, 2016 3 commits
-
-
Spencer Gibb authored
-
Spencer Gibb authored
fixes gh-406
-
Spencer Gibb authored
-
- 05 Feb, 2016 5 commits
-
-
Spencer Gibb authored
fixes gh-813
-
Dave Syer authored
-
Dave Syer authored
-
Dave Syer authored
-
Dave Syer authored
-
- 04 Feb, 2016 3 commits
-
-
Spencer Gibb authored
fixes gh-809
-
Marcin Grzejszczak authored
Upgrading feign
-
Marcin Grzejszczak authored
-
- 03 Feb, 2016 3 commits
-
-
Dave Syer authored
-
Dave Syer authored
-
Dave Syer authored
Also does not rename another one because it fails (see gh-804). The bug can be fixed in feign (https://github.com/Netflix/feign/pull/324), and if that gets merged and released we can revert some of the recent changes to the apache load balancer here.
-
- 02 Feb, 2016 2 commits
-
-
Spencer Gibb authored
fixes gh-807
-
- 01 Feb, 2016 5 commits
-
-
Spencer Gibb authored
Fix typo: zule-core -> zuul-core.
-
Spencer Gibb authored
-
Spencer Gibb authored
-
Spencer Gibb authored
* spectator-issue-264: fix registration bug in SpectatorMetricServices
-
Brian Harrington authored
The primary change is to fix a memory leak reported in Netflix/spectator#264. Each time a gauge was updated it was creating a new registration and because the map holds a strong reference these would never get collected. Further, the aggregate value created by the multiple registrations was not correct. In addition I added some test cases around the various inputs and checked that the results were reflected as expected in the registry. I noticed the timer values had a unit of milliseconds, but it isn't immediately clear if the reported value can ever less than 1.0. The conversion to long is now delayed until after converting to nanoseconds so duration values less than 1.0 will now work instead of just recording 0. For the histogram I changed to just using a cast to `long` to avoid boxing to a `Double`. As an FYI for the future, there is a DoubleDistributionSummary we have experimented with in spectator-ext-sandbox that might be more appropriate for this use-case.
-
- 31 Jan, 2016 2 commits
- 29 Jan, 2016 2 commits
-
-
Dave Syer authored
So that it isn't instantated eagerly on startup by the RefreshScope. Fixes gh-789
-
Niklas Herder authored
-
- 28 Jan, 2016 2 commits
-
-
Dave Syer authored
Tests are very sensitive to the Zuul RequestContext which is thread bound.
-
Marcin Grzejszczak authored
-
- 27 Jan, 2016 4 commits
-
-
Dave Syer authored
-
Dave Syer authored
The Apcahe client does not allow empty body for PATCH, nor does it allow a request to be pre-populated with a Content-Length. Spring Cloud tests were trying to do the former, and Feign allows the latter.
-
Dave Syer authored
-
Adrian Ivan authored
Change SimpleRouteLocator to work with both Zuul and DispatcherServlet Fixes gh-794
-
- 26 Jan, 2016 3 commits
-
-
Dave Syer authored
Fixes gh-710
-
Dave Syer authored
Apparently we already support eureka.datacenter as an alias for archaius.deployment.datacenter and it looks like when that code was written we intended to support eureka.environment in a similar way. But it was never finished off, so this change adds the necessary one line of code. Fixes gh-791
-
Dave Syer authored
Fixes gh-790
-
- 25 Jan, 2016 2 commits
-
-
Dave Syer authored
-
Dave Syer authored
Since DiscoverClient now supports metadata natively, we can use that and not rely on Eureka interafces at all. Fixes gh-787. Also fixes https://github.com/spring-cloud/spring-cloud-config/issues/302
-
- 24 Jan, 2016 1 commit
-
-
Dave Syer authored
Eureka supports native placeholders, but Spring can also be used. Added some docs to show how more explicitly. See gh-761
-