EM-6870 add mimetype update task

[mdayican]

EM-6870 add mimetype update task

[yogesh-hullatti]

Hi @mdayican ,
My comments as below,

1. Remove unused method from src/main/java/uk/gov/hmcts/dm/repository/DocumentContentVersionRepository.java - markMimeTypeUpdated()
2. We have 12 attributes in DocumentContentVersion. We are updating 2 attributes - mimeType & mimeTypeUpdated. Instead of making the DB call to load the entire object. We can use something like below which I believe is efficient and lots of lines of code can be removed related to loading and try/catch.

//Efficient: repository-level update to avoid loading the entity
// In DocumentContentVersionRepository:
@Modifying
@Query("update DocumentContentVersion d set d.mimeType = :mimeType, d.mimeTypeUpdated = true where d.id = :id")
int updateMimeType(@Param("id") UUID id, @Param("mimeType") String mimeType);

[mdayican]

Hi @mdayican , My comments as below,

1. Remove unused method from src/main/java/uk/gov/hmcts/dm/repository/DocumentContentVersionRepository.java - markMimeTypeUpdated()
2. We have 12 attributes in DocumentContentVersion. We are updating 2 attributes - mimeType & mimeTypeUpdated. Instead of making the DB call to load the entire object. We can use something like below which I believe is efficient and lots of lines of code can be removed related to loading and try/catch.

//Efficient: repository-level update to avoid loading the entity
// In DocumentContentVersionRepository:
@Modifying
@Query("update DocumentContentVersion d set d.mimeType = :mimeType, d.mimeTypeUpdated = true where d.id = :id")
int updateMimeType(@Param("id") UUID id, @Param("mimeType") String mimeType);

fixed.

[MichaelJacob21012]

This java comment can be removed

[MichaelJacob21012]

This java comment can be removed

[MichaelJacob21012]

We can update in batch as well similar to

document-management-store-app/src/main/java/uk/gov/hmcts/dm/config/batch/CaseDocumentsDeletionTask.java

Line 100 in 6b9cd5d

int batchCommitSize = 500; // Define the batch size for committing to the DB

[MichaelJacob21012]

missing a unit test for this catch block