2
83%
master: 72%

Ran 02 Feb 2017 04:59AM UTC

Files 379

Run time 16s

Badge

Embed ▾

Committed 02 Feb 2017 02:23AM UTC coverage: 83.085%. First build

Job # 3013.2

Build Type

Pull #1586

travis-ci

Committed by

web-flow

Commit Message

fix race in SubscriberImplTest::receiveMessage

This bug manifests as testBundleAcks deadlocking.

The test
1. publishes a series of messages to a mock server,
2. waits to receive them back,
3. and acknowledges them.

For performance reasons, the client does not immediately
send the acknowledgement request to the server.
Instead, it sends acks in batch every 100ms.
Using a fake clock, the test advances the time by 100ms,
sending the ack-batch, then verify that the mock server
receives the acks.

The bug is in step 2. The test thread waits by waiting on a
CountDownLatch, which is counted down by GRPC thread calling
receiveMessage().
However, the method decrements the latch before acking the message.

On a bad day, the test thread can wake up, advance the clock,
and send the ack-batch before the GRPC thread could add the
message to the batch.
The test then waits for the server to receive an ack it never sent,
deadlocking the test.

The fix is for receiveMessage() to ack the message before
decrementing the counter.

Pull Request Pull Request #1586: fix race in SubscriberImplTest::receiveMessage

Run Details

16789 of 20207 relevant lines covered (83.09%)

161.17 hits per line

GoogleCloudPlatform / google-cloud-java / 3013 / 2
83%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3013.2

GoogleCloudPlatform / google-cloud-java / 3013 / 2 83% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3013.2

GoogleCloudPlatform / google-cloud-java / 3013 / 2
83%
master: 72%

README BADGES
x