This is an automated email from the ASF dual-hosted git repository.

btellier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 5a19a860cc91591d3596790eddb8d66a2f3341f7
Author: Benoit Tellier <[email protected]>
AuthorDate: Fri Jun 4 13:56:48 2021 +0700

    [PERFORMANCE] CassandraMessageDAO should use 
CassandraAsyncExecutor::executeSingleRow
    
    This prevents a single row ResultSet post-traitements to be scheduled
    on the elasticScheduler
---
 .../james/mailbox/cassandra/mail/CassandraMessageDAO.java   | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git 
a/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageDAO.java
 
b/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageDAO.java
index ed3f8c8..1600985 100644
--- 
a/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageDAO.java
+++ 
b/mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageDAO.java
@@ -70,7 +70,6 @@ import 
org.apache.james.mailbox.store.mail.model.impl.PropertyBuilder;
 import com.datastax.driver.core.BoundStatement;
 import com.datastax.driver.core.ConsistencyLevel;
 import com.datastax.driver.core.PreparedStatement;
-import com.datastax.driver.core.ResultSet;
 import com.datastax.driver.core.Row;
 import com.datastax.driver.core.Session;
 import com.datastax.driver.core.UDTValue;
@@ -245,20 +244,14 @@ public class CassandraMessageDAO {
                 .flatMap(resultSet -> message(resultSet, cassandraMessageId, 
fetchType));
     }
 
-    private Mono<ResultSet> retrieveRow(CassandraMessageId messageId) {
-        return cassandraAsyncExecutor.execute(select
+    private Mono<Row> retrieveRow(CassandraMessageId messageId) {
+        return cassandraAsyncExecutor.executeSingleRow(select
             .bind()
             .setUUID(MESSAGE_ID, messageId.get())
             .setConsistencyLevel(consistencyLevel));
     }
 
-    private Mono<MessageRepresentation>
-    message(ResultSet rows, CassandraMessageId cassandraMessageId, FetchType 
fetchType) {
-        if (rows.isExhausted()) {
-            return Mono.empty();
-        }
-
-        Row row = rows.one();
+    private Mono<MessageRepresentation> message(Row row, CassandraMessageId 
cassandraMessageId, FetchType fetchType) {
         BlobId headerId = retrieveBlobId(HEADER_CONTENT, row);
         BlobId bodyId = retrieveBlobId(BODY_CONTENT, row);
         int bodyStartOctet = row.getInt(BODY_START_OCTET);

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to