zhfeng commented on a change in pull request #3681:
URL: https://github.com/apache/camel-quarkus/pull/3681#discussion_r839562609



##########
File path: 
integration-tests/paho/src/main/java/org/apache/camel/quarkus/component/paho/PahoResource.java
##########
@@ -87,4 +88,11 @@ public String mqttExceptionDuringReconnectShouldSucceed() {
         return mqex.getMessage();
     }
 
+    @Path("/readThenWriteWithFilePersistenceShouldSucceed")
+    @GET
+    @Produces(MediaType.TEXT_PLAIN)
+    public String 
readThenWriteWithFilePersistenceShouldSucceed(@QueryParam("message") String 
message) throws Exception {
+        
producerTemplate.requestBody("paho:withFilePersistence?retained=true&cleanSession=false&persistence=FILE",
 message);

Review comment:
       yeah, the ```clientDir``` is empty. I just think if there is the other 
thread opens the directory which could cause the deletion failing. 
unfortunately, ```java.io.File.delete()``` only return ```false``` but we don't 
know the reason. Perhaps  ```java.nio.file.Files.delete(Path)``` can give us 
more details. I will keep investigating. 
   
   Both ```camel-paho``` and ```camel-paho-mqtt5' lack this file persistence 
test, so I think it could be better to open a issue in camel.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to