Coherence Spring Data - When updating don't throw NullPointerException

[coherence-community]

Coherence Spring Data - When updating don't throw NullPointerException

[aseovic]

Yeah, the exception is not the friendliest, because it has occurred during remote lambda invocation, but it is the correct one.

There are no guarantees that the value is not going to be null, so the code should check for that (and typically create a new instance to update):

this.taskRepository.update(id, myModel -> {
        if (myModel == null) {
            myModel = new MyModel(id);
        }
        myModel.setDescription(description);
        return myModel;
    });

Which can also be done using one of the update method overloads, which allow you to specify a value supplier that should be used to create a new value instance when the existing value is null:

this.taskRepository.update(id, MyModel::setDescription, description, MyModel::new);

[ghillert]

@aseovic Just revisiting this a bit more. The relevant code is in AbstractRepositoryBase in Coherence package com.oracle.coherence.repository:

            if (entity == null && factory != null)
                {
                entity = factory.create(entry.getKey());
                }
            updater.update(entity, value);

I would think that if the entity is null and the factory is null, an IllegalStateException should be throw, without calling the updater with a null argument causing the nullpointerexception.

[ghillert]

Added some documentation explain the situation plus work-around.