Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Property REDIS_CONNECTION_STRING seems not working #9

Open
cnneillee opened this issue May 5, 2017 · 0 comments
Open

Property REDIS_CONNECTION_STRING seems not working #9

cnneillee opened this issue May 5, 2017 · 0 comments

Comments

@cnneillee
Copy link

Hi.
REDIS_CONNECTION_STRING is one property of GetRedis. In this project, its description goes that

"The Connection String to use in order to connect to Redis. This is often a comma-separated list of <host>:<port>combinations. For example, host1:2181,host2:2181,host3:2188".

I try to fill this property with "192.168.203.18:6379" but it does not work. When I glance at the source code, I find following things:

@OnScheduled
public void createRedisPool(final ProcessContext context) {
	jedisPool = new JedisPool(new JedisPoolConfig(), context.getProperty(REDIS_CONNECTION_STRING).getValue());
}

Description of this method comes that

[Maven: redis.clients:jedis:2.7.2] redis.clients.jedis JedisPool
public JedisPool (GenericObjectPoolConfig poolConfig, String host)

So in my opinion, this property should be defined in this way —— "Single host, for example 192.168.203.18".

Am I right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant