MongoRepository Save() issue

Rojan_John_V · 2020-11-24T18:49:08.471Z

Hi Team,
Am facing an issue MongoRepository save() method. Once I updated the document by calling save() method and am trying to fetch the same document by using findById(), it will not reflected the updated value in result. could you please help on this issue asap?

Prasad_Saya · 2020-11-25T03:02:32.724Z

Hello @Rojan_John_V, welcome to the MongoDB community forum.

The MongoRepository#save method returns an instance of the saved entity. Can you verify that the returned object has the updated values (you can use a System.out.println(returnedEntity) immediately after the save )?

I tried such code and found querying the same collection using the MongoRepository#.findById returned the document with updated values.

I am using MongoDB v4.2.8 and Spring Boot v2.2.4.

Rojan_John_V · 2020-11-25T03:50:53.321Z

Thanks Prasad for your reply. This issue am facing intermittently and I could see the updated values when I am printing return object from save(). If am putting some time delay between save and findbyid, I could see the expected result.

Prasad_Saya · 2020-11-25T03:59:10.535Z

Rojan_John_V:

This issue am facing intermittently

Are you able to reproduce the issue consistently?

If am putting some time delay between save and findbyid, I could see the expected result.

What is happening if you dont use the time delay?

Rojan_John_V · 2020-11-25T07:30:11.986Z

As per our architecture, the first request will fetch the document by using findbyid() method and ready to update content and save to db. After 200 or 300 milliseconds difference the second request will come and try to fetch the same document, that time I could see the document having old value.
It will happen intermittently not all time. if am setting delay between the requests it never happen, without delay it will happen sometimes.
I tried to reproduce this issue in local mongo db by triggering bulk of requests, but I could not see this issue. In dev region I could see this with concurrent requests.
In dev we are using MongoAtlas with one primary node and two secondary node.

Prasad_Saya · 2020-11-25T08:02:43.718Z

What are your settings about the Read Preference?

Rojan_John_V · 2020-11-25T10:36:40.791Z

We are not setting any specific readPreference value, I believe the default should be Primary. How can we check this in MongoAtlas console?

Rojan_John_V · 2020-11-25T10:36:42.868Z

The connection url i like this
mongodb+srv://{USER}:{PASSWORD}@{HOST}/{DB_NAME}?retryWrites=true&w=majority
do we need to set readPreference along with the connection url ?

Prasad_Saya · 2020-11-25T11:05:10.255Z

Rojan_John_V:

We are not setting any specific readPreference value, I believe the default should be Primary

The default Read Preference is Primary. Having a setting other than Primary, can result in reading stale data. Read Preference can be set within connection URI or the Java / Spring Boot application.

Rojan_John_V · 2020-11-25T12:12:12.459Z

If the default read preference is Primary, we don’t need to set the read preference specifically. do you have any other suggestions?

Prasad_Saya · 2020-11-25T12:37:18.621Z

I suggest check your application code if you are setting at the reads you are doing (for this specific read functionality).

Further, I think you can tell more about your application and the code you are using for this functionality:

As per our architecture, the first request will fetch the document by using findbyid() method and ready to update content and save to db. After 200 or 300 milliseconds difference the second request will come and try to fetch the same document, that time I could see the document having old value. It will happen intermittently not all time. if am setting delay between the requests it never happen, without delay it will happen sometimes.

Rojan_John_V · 2020-11-26T11:34:59.134Z

Do you have any suggestion to set read and write concern in connection url. As of now the application using below connection url
The connection url :
mongodb+srv://{USER}:{PASSWORD}@{HOST}/{DB_NAME}?retryWrites=true&w=majority

Samuel_Parra · 2021-11-26T20:24:16.052Z

Hi @Rojan_John_V, Did you find a solution? I am facing the same issue.

Santhosh_Voodem · 2021-12-10T08:44:52.623Z

Any updated on this issue. We are also facing same issue. I need some guide to fix/solution for this issue.

Nait_Belkacem_Youssef · 2022-08-19T10:21:06.841Z

I’m having the same issue here, any news ?

creeper12356_N_A · 2024-12-28T13:28:20.681Z

I encountered the same issue TODAY.
My element class is :

@Data
@Document(collection = "comment")
public class Comment {
    @Id
    private String id;
    private Integer bookId;
    private String username;
    private String content;
    private Date time;
    private List<Comment> replies;
    private String replyTo;

    public Comment() {
        // not used
    }
    public Comment(Integer bookId, String username, String content, String reply_to) {
        this.bookId = bookId;
        this.username = username;
        this.content = content;
        this.time = new Date();
        this.replies = new ArrayList<>();
        this.replyTo = reply_to;
    }
}

Initially, I tried to updating a comment’s reply list using MongoRepository.save method. It, however , doesn’t update anything.
Then, I referenced this post and manually write a updateComment method to update:

@Repository
public interface BookCommentRepository extends MongoRepository<Comment, String> {
    List<Comment> findByBookId(Integer bookId);

    @Query("{ 'id': ?0 }")
    @Update("{ '$set': {'replies': ?1 }}")
    void updateComments(String id, List<Comment> comments);
}

It does works.

saq_rio · 2025-01-13T00:11:15.635Z

Rojan_John_V:

Hi Team,
Am facing an issue MongoRepository save() method. Once I updated the document by calling save() method and am trying to fetch the same document by using findById(), it will macdonalds philippines not reflected the updated value in result. could you please help on this issue asap?

It seems like the update isn’t being reflected immediately. Try checking the return value of save() and ensure there’s no caching issue. Also, ensure the update is committed before calling findById().