Skip to content

Commit f4d4d5f

Browse files
committed
Merge pull request #179 from NiteshKant/master
Fixed: monitors are not registered with servo
2 parents f86dd0b + 7fedd56 commit f4d4d5f

File tree

2 files changed

+22
-2
lines changed

2 files changed

+22
-2
lines changed

CHANGES.md

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,29 @@
11
# RxNetty Releases #
22

33

4+
### Version 0.3.9 ###
5+
6+
[Milestone](https://github.com/Netflix/RxNetty/issues?milestone=7&state=closed)
7+
8+
* [Pull 179] (https://github.com/Netflix/RxNetty/issues/179) Monitors were not getting registered with servo
9+
10+
Artifacts: [Maven Central](http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22com.netflix.rxnetty%22%20AND%20v%3A%220.3.9%22)
11+
12+
### Version 0.3.8 ###
13+
14+
[Milestone](https://github.com/Netflix/RxNetty/issues?milestone=5&state=closed)
15+
16+
* [Issue 150] (https://github.com/Netflix/RxNetty/issues/150) Removed deprecated PoolStats.
17+
* [Issue 169] (https://github.com/Netflix/RxNetty/issues/169) Removed ContentSource in favor of Observable for HttpClientRequest
18+
* [Issue 172] (https://github.com/Netflix/RxNetty/issues/172) Missing connection pool events from client built by RxContexts constructs
19+
* [Issue 175] (https://github.com/Netflix/RxNetty/issues/175) Optionally disable System time calls for metrics
20+
* [Issue 177] (https://github.com/Netflix/RxNetty/issues/177) For HTTP server use channelReadComplete() to flush writes
21+
22+
Artifacts: [Maven Central](http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22com.netflix.rxnetty%22%20AND%20v%3A%220.3.8%22)
23+
424
### Version 0.3.7 ###
525

6-
[Milestone](https://github.com/Netflix/RxNetty/issues?milestone=2&state=closed)
26+
[Milestone](https://github.com/Netflix/RxNetty/issues?milestone=4&state=closed)
727

828
* [Issue 98] (https://github.com/Netflix/RxNetty/issues/98) Added pluggable metrics infrastructure with rx-netty-servo implementation.
929
* [Issue 106] (https://github.com/Netflix/RxNetty/issues/106) Added TLS support for TCP & HTTP (HTTPS)

rx-netty-servo/src/main/java/io/reactivex/netty/servo/RefCountingMonitor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public void onCompleted(Object monitor) {
4343
}
4444

4545
public void onSubscribe(Object monitor) {
46-
if (subscriptionCount.incrementAndGet() == 0) {
46+
if (subscriptionCount.getAndIncrement() == 0) {
4747
ServoUtils.registerObject(monitorId, monitor);
4848
}
4949
}

0 commit comments

Comments
 (0)