search service for full text search implementation;

search bean configuration

5 files changed, 166 insertions, 34 deletions

diff --git a/juick-server/src/main/java/com/juick/configuration/SearchConfiguration.java b/juick-server/src/main/java/com/juick/configuration/SearchConfiguration.java
new file mode 100644
index 00000000..40473433
--- /dev/null
+++ b/juick-server/src/main/java/com/juick/configuration/SearchConfiguration.java
@@ -0,0 +1,54 @@
+package com.juick.configuration;
+
+import com.juick.service.search.SearchService;
+import com.juick.service.search.SphinxSearchServiceImpl;
+import org.apache.commons.dbcp2.BasicDataSource;
+import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Configuration;
+import org.springframework.context.annotation.PropertySource;
+import org.springframework.core.env.Environment;
+
+import javax.annotation.Resource;
+import java.util.Collections;
+import java.util.List;
+
+/**
+ * Created by aalexeev on 11/18/16.
+ */
+@Configuration
+@PropertySource(value = {"classpath:juick.conf"})
+public class SearchConfiguration {
+    @Resource
+    private Environment env;
+
+    @Bean
+    public SearchService searchService() {
+        BasicDataSource dataSource = new BasicDataSource();
+
+        dataSource.setDriverClassName(env.getProperty("sphinx_driver", "com.mysql.jdbc.Driver"));
+        dataSource.setUrl(env.getProperty("sphinx_url", "jdbc:mysql://127.0.0.1:9306?autoReconnect=true&useUnicode=yes&characterEncoding=utf8&maxAllowedPacket=512000"));
+        dataSource.setUsername(env.getProperty("sphinx_user", ""));
+        dataSource.setPassword(env.getProperty("sphinx_password", ""));
+
+        return new SphinxSearchServiceImpl(dataSource);
+    }
+
+    @Bean
+    public SearchService emptySearchService() {
+        return new SearchService() {
+            @Override
+            public void setMaxResult(int maxResult) {
+            }
+
+            @Override
+            public List<Integer> searchInAllMessages(String searchString, int messageIdBefore) {
+                return Collections.emptyList();
+            }
+
+            @Override
+            public List<Integer> searchByStringAndUser(String searchString, int userId, int messageIdBefore) {
+                return Collections.emptyList();
+            }
+        };
+    }
+}
diff --git a/juick-server/src/main/java/com/juick/service/MessagesService.java b/juick-server/src/main/java/com/juick/service/MessagesService.java
index 83ab0dce..5990f408 100644
--- a/juick-server/src/main/java/com/juick/service/MessagesService.java
+++ b/juick-server/src/main/java/com/juick/service/MessagesService.java
@@ -58,7 +58,7 @@ public interface MessagesService {
 
     List<Integer> getPhotos(int visitorUid, int before);
 
-    List<Integer> getSearch(JdbcTemplate sqlSearch, String search, int before);
+    List<Integer> getSearch(String search, int before);
 
     List<Integer> getUserBlog(int uid, int privacy, int before);
 
diff --git a/juick-server/src/main/java/com/juick/service/MessagesServiceImpl.java b/juick-server/src/main/java/com/juick/service/MessagesServiceImpl.java
index 6fc71ac3..6f33bbf3 100644
--- a/juick-server/src/main/java/com/juick/service/MessagesServiceImpl.java
+++ b/juick-server/src/main/java/com/juick/service/MessagesServiceImpl.java
@@ -4,6 +4,7 @@ import com.juick.Message;
 import com.juick.Tag;
 import com.juick.User;
 import com.juick.server.helpers.PrivacyOpts;
+import com.juick.service.search.SearchService;
 import com.juick.util.MessageUtils;
 import org.apache.commons.collections4.CollectionUtils;
 import org.apache.commons.lang3.StringEscapeUtils;
@@ -30,13 +31,17 @@ import java.util.*;
 @Repository
 public class MessagesServiceImpl extends BaseJdbcService implements MessagesService {
     private final UserService userService;
+    private final SearchService searchService;
 
     @Inject
-    public MessagesServiceImpl(JdbcTemplate jdbcTemplate, UserService userService) {
+    public MessagesServiceImpl(JdbcTemplate jdbcTemplate, UserService userService, SearchService searchService) {
         super(jdbcTemplate, null);
 
         Assert.notNull(userService);
         this.userService = userService;
+
+        Assert.notNull(searchService);
+        this.searchService = searchService;
     }
 
     private class MessageMapper implements RowMapper<Message> {
@@ -576,26 +581,14 @@ public class MessagesServiceImpl extends BaseJdbcService implements MessagesServ
 
     @Transactional(readOnly = true)
     @Override
-    public List<Integer> getSearch(final JdbcTemplate sqlSearch, final String search, final int before) {
-        List<Integer> mids;
-
-        if (before > 0) {
-            mids = sqlSearch.queryForList(
-                    "SELECT id AS message_id FROM messages WHERE MATCH(?) AND id < ? ORDER BY id DESC LIMIT 25",
-                    Integer.class,
-                    search, before);
-        } else {
-            mids = sqlSearch.queryForList(
-                    "SELECT id AS message_id FROM messages WHERE MATCH(?) ORDER BY id DESC LIMIT 25",
-                    Integer.class,
-                    search);
-        }
-        if (mids.size() > 0) {
+    public List<Integer> getSearch(final String search, final int before) {
+        List<Integer> mids = searchService.getMessageIdsInAllMessages(search, before);
+
+        if (!mids.isEmpty())
             return getNamedParameterJdbcTemplate().queryForList(
                     "SELECT message_id FROM messages WHERE message_id IN (:ids) AND privacy > 0 ORDER BY message_id DESC LIMIT 20",
                     new MapSqlParameterSource("ids", mids),
                     Integer.class);
-        }
         return mids;
     }
 
@@ -672,23 +665,9 @@ public class MessagesServiceImpl extends BaseJdbcService implements MessagesServ
     @Transactional(readOnly = true)
     @Override
     public List<Integer> getUserSearch(final JdbcTemplate sqlSearch, final int UID, final String search, final int privacy, final int before) {
-        List<Integer> mids;
-
-        if (before > 0) {
-            mids = sqlSearch.queryForList(
-                    "SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) AND id<? " +
-                            "ORDER BY id DESC LIMIT 20",
-                    new Object[]{UID, search, before},
-                    Integer.class);
-        } else {
-            mids = sqlSearch.queryForList(
-                    "SELECT id AS message_id FROM messages WHERE user_id=? AND MATCH(?) " +
-                            "ORDER BY id DESC LIMIT 20",
-                    new Object[]{UID, search},
-                    Integer.class);
-        }
+        List<Integer> mids = searchService.getMessageIdsByUser(search, UID, before);
 
-        if (mids.size() > 0) {
+        if (!mids.isEmpty()) {
             return getNamedParameterJdbcTemplate().queryForList(
                     "SELECT message_id FROM messages WHERE message_id IN (:ids) AND privacy >= :privacy ORDER BY message_id DESC",
                     new MapSqlParameterSource("ids", mids)
diff --git a/juick-server/src/main/java/com/juick/service/search/SearchService.java b/juick-server/src/main/java/com/juick/service/search/SearchService.java
new file mode 100644
index 00000000..21deb0b1
--- /dev/null
+++ b/juick-server/src/main/java/com/juick/service/search/SearchService.java
@@ -0,0 +1,14 @@
+package com.juick.service.search;
+
+import java.util.List;
+
+/**
+ * Created by aalexeev on 11/18/16.
+ */
+public interface SearchService {
+    void setMaxResult(int maxResult);
+
+    List<Integer> searchInAllMessages(String searchString, int messageIdBefore);
+
+    List<Integer> searchByStringAndUser(String searchString, final int userId, int messageIdBefore);
+}
diff --git a/juick-server/src/main/java/com/juick/service/search/SphinxSearchServiceImpl.java b/juick-server/src/main/java/com/juick/service/search/SphinxSearchServiceImpl.java
new file mode 100644
index 00000000..de0a101f
--- /dev/null
+++ b/juick-server/src/main/java/com/juick/service/search/SphinxSearchServiceImpl.java
@@ -0,0 +1,85 @@
+package com.juick.service.search;
+
+import org.apache.commons.lang3.StringUtils;
+import org.springframework.jdbc.core.JdbcTemplate;
+import org.springframework.jdbc.core.namedparam.MapSqlParameterSource;
+import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate;
+import org.springframework.util.Assert;
+
+import javax.sql.DataSource;
+import java.util.Collections;
+import java.util.List;
+
+/**
+ * Created by aalexeev on 11/18/16.
+ */
+
+/* Note
+ * Don't use any spring's component annotation (such as @Repository, @Service, @Component, etc).
+ * This class directly used by spring's search configuration class
+ */
+public class SphinxSearchServiceImpl implements SearchService {
+    private static final int DEFAULT_MAX_RESULT = 25;
+
+    private final NamedParameterJdbcTemplate namedParameterSearchJdbcTemplate;
+
+    private int maxResult = DEFAULT_MAX_RESULT;
+
+
+    public SphinxSearchServiceImpl(JdbcTemplate searchJdbcTemplate) {
+        Assert.notNull(searchJdbcTemplate);
+        this.namedParameterSearchJdbcTemplate = new NamedParameterJdbcTemplate(searchJdbcTemplate);
+    }
+
+    public SphinxSearchServiceImpl(DataSource searchDataSource) {
+        Assert.notNull(searchDataSource);
+        this.namedParameterSearchJdbcTemplate = new NamedParameterJdbcTemplate(searchDataSource);
+    }
+
+    @Override
+    public List<Integer> searchInAllMessages(final String searchString, final int messageIdBefore) {
+        if (StringUtils.isBlank(searchString))
+            return Collections.emptyList();
+
+        MapSqlParameterSource sqlParameterSource = new MapSqlParameterSource()
+                .addValue("search", searchString)
+                .addValue("before", messageIdBefore)
+                .addValue("limit", maxResult);
+
+        return namedParameterSearchJdbcTemplate.queryForList(
+                "SELECT id AS message_id FROM messages WHERE MATCH(:search) " +
+                        (messageIdBefore > 0 ?
+                                " AND id < :before " : "") +
+                        " ORDER BY id DESC LIMIT :limit",
+                sqlParameterSource,
+                Integer.class);
+    }
+
+    @Override
+    public List<Integer> searchByStringAndUser(final String searchString, final int userId, int messageIdBefore) {
+        if (StringUtils.isBlank(searchString))
+            return Collections.emptyList();
+
+        MapSqlParameterSource sqlParameterSource = new MapSqlParameterSource()
+                .addValue("search", searchString)
+                .addValue("userId", userId)
+                .addValue("before", messageIdBefore)
+                .addValue("limit", maxResult);
+
+        return namedParameterSearchJdbcTemplate.queryForList(
+                "SELECT id AS message_id FROM messages WHERE user_id = :userId AND MATCH(:search) " +
+                        (messageIdBefore > 0 ?
+                                " AND id < :before " : "") +
+                        " ORDER BY id DESC LIMIT :limit",
+                sqlParameterSource,
+                Integer.class);
+    }
+
+    @Override
+    public void setMaxResult(int maxResult) {
+        if (maxResult <= 0)
+            throw new IllegalArgumentException("maxResult value (" + maxResult + ") must be greater then 0");
+
+        this.maxResult = maxResult;
+    }
+}
\ No newline at end of file