ignite-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [ignite] pavlukhin commented on a change in pull request #6917: IGNITE-12189 Implementation correct limit for TextQuery
Date Wed, 23 Oct 2019 13:39:28 GMT
pavlukhin commented on a change in pull request #6917: IGNITE-12189 Implementation correct
limit for TextQuery
URL: https://github.com/apache/ignite/pull/6917#discussion_r338015321
 
 

 ##########
 File path: modules/core/src/main/java/org/apache/ignite/cache/query/TextQuery.java
 ##########
 @@ -144,6 +173,26 @@ public String getText() {
         return this;
     }
 
+    /**
+     * Gets limit to response records count.
+     *
+     * @return Limit value.
+     */
+    public int getLimit() {
+        return limit;
+    }
+
+    /**
+     * Sets limit to response records count.
+     *
+     * @param limit If 0 or less, the limit considered to be Integer.MAX_VALUE, that is virtually
no limit.
+     * @return {@code this} For chaining.
+     */
+    public TextQuery<K, V> setLimit(int limit) {
+        this.limit = limit;
 
 Review comment:
   For code style consistency (check other setters) there should be empty line between this
and a next statement.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message