You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ol...@apache.org on 2016/04/09 09:21:33 UTC

svn commit: r1738327 - in /sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail: pom.xml src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java

Author: olli
Date: Sat Apr  9 07:21:33 2016
New Revision: 1738327

URL: http://svn.apache.org/viewvc?rev=1738327&view=rev
Log:
SLING-5644 Provide an messaging implementation based on Commons Email

wrap ThreadPool in Executor to stick with Commons Threads 3.2.6

Modified:
    sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/pom.xml
    sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java

Modified: sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/pom.xml
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/pom.xml?rev=1738327&r1=1738326&r2=1738327&view=diff
==============================================================================
--- sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/pom.xml (original)
+++ sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/pom.xml Sat Apr  9 07:21:33 2016
@@ -165,7 +165,7 @@
     <dependency>
       <groupId>org.apache.sling</groupId>
       <artifactId>org.apache.sling.commons.threads</artifactId>
-      <version>3.2.7-SNAPSHOT</version>
+      <version>3.2.6</version>
       <scope>provided</scope>
     </dependency>
     <dependency>

Modified: sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java?rev=1738327&r1=1738326&r2=1738327&view=diff
==============================================================================
--- sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java (original)
+++ sling/trunk/bundles/commons/org.apache.sling.commons.messaging.mail/src/main/java/org/apache/sling/commons/messaging/mail/internal/SimpleMailService.java Sat Apr  9 07:21:33 2016
@@ -114,7 +114,7 @@ public class SimpleMailService implement
 
     @Override
     public CompletableFuture<Result> send(@Nonnull final String message, @Nonnull final String recipient, @Nonnull final Map data) {
-        return CompletableFuture.supplyAsync(() -> sendMail(message, recipient, data, mailBuilder), threadPool);
+        return CompletableFuture.supplyAsync(() -> sendMail(message, recipient, data, mailBuilder), runnable -> threadPool.submit(runnable));
     }
 
     private MailResult sendMail(final String message, final String recipient, final Map data, final MailBuilder mailBuilder) {