Support server array constructor #6

byplayer · 2017-10-12T10:13:43Z

Support server array constructor
Fix connection error issue when pooling socket is closed by server
Fix NullPointer exception when call roma function before initialize routingData

If pooled socket expired by ROMA server, socket is closed, but client still has that connection. In that case, client received error. So I checked exception, and catch it, then retry once.

When routing isn't created, then null pointer exception occur

When getConnection is called before initialize routingData, target function throws null pointer exception.

When Pooled connection are closed by server, this method return exception. So add retry logic for it.

Change test ROMA port 11211,11311 to 11311,11411, because 11211 port is unstable on travis CI.

byplayer and others added 17 commits July 25, 2017 12:50

add gtags files into ignore list

512cfe7

support String[] of RomaClient constructor

15f69b1

fix typo

f3a59b8

clean up key for test

89262ae

change test name more meaning

60d5111

fix dead node issue for initialize

2997499

Fix send commands error when socket is closed

fa156f6

If pooled socket expired by ROMA server, socket is closed, but client still has that connection. In that case, client received error. So I checked exception, and catch it, then retry once.

add test section

b0c5cd2

bump up version

1e5f43d

fix null pointer exception

2b7cb43

When routing isn't created, then null pointer exception occur

Fix crash issue. When getConnection is called before initialize

9b0d11d

When getConnection is called before initialize routingData, target function throws null pointer exception.

fix retry logic, when socket exception has occurred

7750d74

When Pooled connection are closed by server, this method return exception. So add retry logic for it.

fix default retry count

3992d5c

Merge branch 'master' into feature/array_constructor

d09b23f

Merge branch 'use_travis_ci' into feature/array_constructor

2a0c533

change test ROMA port

0c224a2

Change test ROMA port 11211,11311 to 11311,11411, because 11211 port is unstable on travis CI.

Merge branch 'master' into feature/array_constructor

3dab016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Support server array constructor #6

Support server array constructor #6

byplayer commented Oct 12, 2017

Support server array constructor #6

Are you sure you want to change the base?

Support server array constructor #6

Conversation

byplayer commented Oct 12, 2017