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.

Sign up for GitHub

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

Jump to bottom

NPE on ClientExecutionPoolSizeLowTest.testOwnerNodeTerminateWithAsyncOperations() #6853

Closed
eminn opened this issue Nov 24, 2015 · 0 comments · Fixed by #6936
Closed

NPE on ClientExecutionPoolSizeLowTest.testOwnerNodeTerminateWithAsyncOperations() #6853

eminn opened this issue Nov 24, 2015 · 0 comments · Fixed by #6936
Assignees
@sancar
Labels
Source: Internal PR or issue was opened by an employee Team: Client Type: Defect
Milestone
3.6

Comments

@eminn
Copy link
Contributor

eminn commented Nov 24, 2015

The test method is logging the following output on the new client module

13:00:46,742 ERROR [Invocation] hz._hzInstance_2_dev.async.thread-4 - [127.0.0.1]:5002 [dev] [3.6-RC2-SNAPSHOT] Failed asynchronous execution of execution callback: com.hazelcast.client.impl.protocol.task.map.MapPutMessageTask@46decc20for call Invocation{serviceName='hz:impl:mapService', op=com.hazelcast.map.impl.operation.PutOperation{serviceName='hz:impl:mapService', partitionId=87, replicaIndex=0, callId=0, invocationTime=1448362845673 (Tue Nov 24 13:00:45 EET 2015), waitTimeout=-1, callTimeout=60000, name=0ab1c75e-58e0-4c34-afb9-3031adc7f3fa, name=0ab1c75e-58e0-4c34-afb9-3031adc7f3fa}, partitionId=87, replicaIndex=0, tryCount=250, tryPauseMillis=500, invokeCount=1, callTimeout=60000, target=Address[127.0.0.1]:5002, backupsExpected=1, backupsCompleted=0, connection=MockConnection{localEndpoint=Address[127.0.0.1]:5002, remoteEndpoint=Address[127.0.0.1]:5002}}
java.lang.NullPointerException
    at com.hazelcast.client.impl.protocol.task.map.AbstractMapPutMessageTask.beforeResponse(AbstractMapPutMessageTask.java:47)
    at com.hazelcast.client.impl.protocol.task.AbstractPartitionMessageTask.onFailure(AbstractPartitionMessageTask.java:79)
    at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture$1.run(InvocationFuture.java:129)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
    at java.lang.Thread.run(Thread.java:745)
    at com.hazelcast.util.executor.HazelcastManagedThread.executeRun(HazelcastManagedThread.java:76)
    at com.hazelcast.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:92)
@eminn eminn added this to the 3.6 milestone Nov 24, 2015
@eminn eminn changed the title NPE on com.hazelcast.client.io.ClientExecutionPoolSizeLowTest.testOwnerNodeTerminateWithAsyncOperations() NPE on ClientExecutionPoolSizeLowTest.testOwnerNodeTerminateWithAsyncOperations() Nov 24, 2015
@eminn eminn mentioned this issue Nov 24, 2015
Closed
@sancar sancar mentioned this issue Dec 1, 2015
Merged
@sancar sancar assigned sancar and unassigned eminn Dec 1, 2015
sancar pushed a commit to sancar/hazelcast that referenced this issue Dec 2, 2015
Fixes NPE on ClientExecutionPoolSizeLowTest
6cf3e64 
pulled service access logic to nodeEngine in order not to return
null from getService()

move listener initilization from node constructor to start. Add start
to quorum service and move listener registration there

Fixes hazelcast#6853
Backport of hazelcast#6936
@sancar sancar mentioned this issue Dec 2, 2015
Merged
sancar pushed a commit to sancar/hazelcast that referenced this issue Dec 3, 2015
Fixes NPE on ClientExecutionPoolSizeLowTest
1eaa9ba 
pulled service access logic to nodeEngine in order not to return
null from getService()

move listener initilization from node constructor to start. Add start
to quorum service and move listener registration there

Fixes hazelcast#6853
Backport of hazelcast#6936
sancar pushed a commit to sancar/hazelcast that referenced this issue Dec 3, 2015
Fixes NPE on ClientExecutionPoolSizeLowTest
92bcf0f 
pulled service access logic to nodeEngine in order not to return
null from getService()

move listener initilization from node constructor to start. Add start
to quorum service and move listener registration there

Fixes hazelcast#6853
Backport of hazelcast#6936
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sancar sancar

Labels
Source: Internal PR or issue was opened by an employee Team: Client Type: Defect
Projects
None yet
Milestone
3.6
3 participants
@eminn @sancar @mmedenjak