[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3681: paho: Add test coverage for file persistence #3680

[GitBox <gi...@apache.org>]

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



##########
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:
       Indeed, I have the same with mqtt3. In native with cleanSession = true, there is a NullPointerException. So far I've been able to investigate at this stage, it appears that the clientDir is deleted.
   
   The deletion show up only in native mode, but I suspect it is a more general race condition. Perharps even in camel-paho itself. At this stage, I was not able to fully qualify this bug. cleanSession = false allow to test with file persistence without this bug.




-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org