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

NPE when issuing multi-exec against cluster #187

Closed
rovarghe opened this issue Feb 3, 2016 · 1 comment
Closed

NPE when issuing multi-exec against cluster #187

rovarghe opened this issue Feb 3, 2016 · 1 comment
Labels
type: bug A general bug

Comments

@rovarghe
Copy link

rovarghe commented Feb 3, 2016

Test case: (4.1.Final)

        RedisClusterClient rcc = RedisClusterClient.create(uris);
        cconn = rcc.connect();
        RedisAdvancedClusterCommands ccommands = cconn.sync();

        ccommands.multi();

        ccommands.set("one", "1");
        ccommands.set("two", "2");
        ccommands.mget("one", "two");
        ccommands.lpush("list", "11");
        ccommands.llen("list");

        ccommands.exec();

Throws:
java.lang.NullPointerException
at com.lambdaworks.redis.cluster.ClusterDistributionChannelWriter.write(ClusterDistributionChannelWriter.java:99)
at com.lambdaworks.redis.RedisChannelHandler.dispatch(RedisChannelHandler.java:106)
at com.lambdaworks.redis.cluster.StatefulRedisClusterConnectionImpl.dispatch(StatefulRedisClusterConnectionImpl.java:189)
at com.lambdaworks.redis.AbstractRedisAsyncCommands.dispatch(AbstractRedisAsyncCommands.java:1726)
at com.lambdaworks.redis.AbstractRedisAsyncCommands.dispatch(AbstractRedisAsyncCommands.java:1721)
at com.lambdaworks.redis.AbstractRedisAsyncCommands.dispatch(AbstractRedisAsyncCommands.java:1717)
at com.lambdaworks.redis.AbstractRedisAsyncCommands.exec(AbstractRedisAsyncCommands.java:351)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at com.lambdaworks.redis.cluster.StatefulRedisClusterConnectionImpl$ClusterFutureSyncInvocationHandler.handleInvocation(StatefulRedisClusterConnectionImpl.java:334)
at com.google.common.reflect.AbstractInvocationHandler.invoke(AbstractInvocationHandler.java:87)
at com.sun.proxy.$Proxy4.exec(Unknown Source)

@mp911de mp911de added the type: bug A general bug label Feb 3, 2016
@mp911de
Copy link
Collaborator

mp911de commented Feb 4, 2016

Fixed NPE but this won't fix the problem of transaction coordination. I posted my considerations about Redis Cluster Transactions as they could be seen from a high-level API to https://www.reddit.com/r/redis/comments/445u9e/redis_cluster_transactions/

@mp911de mp911de closed this as completed Feb 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

2 participants