javadocs fixed

redisson/src/main/java/org/redisson/CronSchedule.java

@@ -39,8 +39,8 @@ public class CronSchedule {
     /**
      * Creates cron expression object with defined expression string
      * 
-     * @param expression
-     * @return
+     * @param expression of cron
+     * @return object
      * @throws IllegalArgumentException
      *             wrapping a ParseException if the expression is invalid
      */
@@ -52,9 +52,9 @@ public static CronSchedule of(String expression) {
      * Creates cron expression which schedule task execution
      * every day at the given time 
      * 
-     * @param hour
-     * @param minute
-     * @return
+     * @param hour of schedule
+     * @param minute of schedule
+     * @return object
      * @throws IllegalArgumentException
      *             wrapping a ParseException if the expression is invalid
      */
@@ -68,18 +68,10 @@ public static CronSchedule dailyAtHourAndMinute(int hour, int minute) {
      * every given days of the week at the given time.
      * Use Calendar object constants to define day.
      * 
-     * @see Calendar#SUNDAY
-     * @see Calendar#MONDAY
-     * @see Calendar#TUESDAY
-     * @see Calendar#WEDNESDAY
-     * @see Calendar#THURSDAY
-     * @see Calendar#FRIDAY
-     * @see Calendar#SATURDAY 
-     * 
-     * @param hour
-     * @param minute
+     * @param hour of schedule
+     * @param minute of schedule
      * @param daysOfWeek - Calendar object constants
-     * @return
+     * @return object
      */
     public static CronSchedule weeklyOnDayAndHourAndMinute(int hour, int minute, Integer... daysOfWeek) {
         if (daysOfWeek == null || daysOfWeek.length == 0) {
@@ -98,10 +90,10 @@ public static CronSchedule weeklyOnDayAndHourAndMinute(int hour, int minute, Int
      * Creates cron expression which schedule task execution
      * every given day of the month at the given time
      * 
-     * @param hour
-     * @param minute
-     * @param daysOfWeek
-     * @return
+     * @param hour of schedule
+     * @param minute of schedule
+     * @param dayOfMonth of schedule
+     * @return object
      */
     public static CronSchedule monthlyOnDayAndHourAndMinute(int dayOfMonth, int hour, int minute) {
         String expression = String.format("0 %d %d %d * ?", minute, hour, dayOfMonth);

redisson/src/main/java/org/redisson/PubSubMessageListener.java

@@ -23,8 +23,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <K>
- * @param <V>
+ * @param <V> value
  */
 public class PubSubMessageListener<V> implements RedisPubSubListener<Object> {
 

redisson/src/main/java/org/redisson/PubSubPatternMessageListener.java

@@ -15,7 +15,6 @@
  */
 package org.redisson;
 
-import org.redisson.api.listener.MessageListener;
 import org.redisson.api.listener.PatternMessageListener;
 import org.redisson.client.RedisPubSubListener;
 import org.redisson.client.protocol.pubsub.PubSubType;
@@ -24,8 +23,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <K>
- * @param <V>
+ * @param <V> value
  */
 public class PubSubPatternMessageListener<V> implements RedisPubSubListener<V> {
 

redisson/src/main/java/org/redisson/PubSubPatternStatusListener.java

@@ -23,8 +23,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <K>
- * @param <V>
+ * @param <V> value
  */
 public class PubSubPatternStatusListener<V> implements RedisPubSubListener<V> {
 

redisson/src/main/java/org/redisson/PubSubStatusListener.java

@@ -23,8 +23,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <K>
- * @param <V>
+ * @param <V> value
  */
 public class PubSubStatusListener<V> implements RedisPubSubListener<V> {
 

redisson/src/main/java/org/redisson/Redisson.java

@@ -135,7 +135,7 @@ public static RedissonClient create() {
     /**
      * Create sync/async Redisson instance with provided config
      *
-     * @param config
+     * @param config for Redisson
      * @return Redisson instance
      */
     public static RedissonClient create(Config config) {
@@ -163,6 +163,7 @@ public static RedissonReactiveClient createReactive() {
     /**
      * Create reactive Redisson instance with provided config
      *
+     * @param config for Redisson
      * @return Redisson instance
      */
     public static RedissonReactiveClient createReactive(Config config) {

redisson/src/main/java/org/redisson/RedissonBloomFilter.java

@@ -45,7 +45,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <T>
+ * @param <T> type of object
  */
 public class RedissonBloomFilter<T> extends RedissonExpirable implements RBloomFilter<T> {
 

redisson/src/main/java/org/redisson/RedissonGeo.java

@@ -47,7 +47,7 @@
  * 
  * @author Nikita Koksharov
  *
- * @param <V>
+ * @param <V> value
  */
 public class RedissonGeo<V> extends RedissonExpirable implements RGeo<V> {
 

redisson/src/main/java/org/redisson/RedissonHyperLogLog.java

@@ -30,7 +30,7 @@
  * 
  * @author Nikita Koksharov
  *
- * @param <V>
+ * @param <V> value
  */
 public class RedissonHyperLogLog<V> extends RedissonExpirable implements RHyperLogLog<V> {
 

redisson/src/main/java/org/redisson/RedissonMultiLock.java

@@ -46,7 +46,7 @@ public class RedissonMultiLock implements Lock {
      * Creates instance with multiple {@link RLock} objects.
      * Each RLock object could be created by own Redisson instance.
      *
-     * @param locks
+     * @param locks - array of locks
      */
     public RedissonMultiLock(RLock... locks) {
         if (locks.length == 0) {

redisson/src/main/java/org/redisson/RedissonNode.java

@@ -170,7 +170,7 @@ private void retrieveAdresses() {
     /**
      * Create Redisson node instance with provided config
      *
-     * @param config
+     * @param config of RedissonNode
      * @return RedissonNode instance
      */
     public static RedissonNode create(RedissonNodeConfig config) {
@@ -180,8 +180,8 @@ public static RedissonNode create(RedissonNodeConfig config) {
     /**
      * Create Redisson node instance with provided config and Redisson instance
      *
-     * @param config
-     * @param redisson
+     * @param config of RedissonNode
+     * @param redisson instance
      * @return RedissonNode instance
      */
     public static RedissonNode create(RedissonNodeConfig config, Redisson redisson) {

redisson/src/main/java/org/redisson/RedissonRedLock.java

@@ -38,7 +38,7 @@ public class RedissonRedLock extends RedissonMultiLock {
      * Creates instance with multiple {@link RLock} objects.
      * Each RLock object could be created by own Redisson instance.
      *
-     * @param locks
+     * @param locks - array of locks
      */
     public RedissonRedLock(RLock... locks) {
         super(locks);

redisson/src/main/java/org/redisson/RedissonSetCache.java

@@ -15,7 +15,6 @@
  */
 package org.redisson;
 
-import java.io.IOException;
 import java.net.InetSocketAddress;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -37,11 +36,9 @@
 import org.redisson.client.protocol.decoder.ListScanResult;
 import org.redisson.command.CommandAsyncExecutor;
 
-import io.netty.util.concurrent.Future;
-
 /**
  * <p>Set-based cache with ability to set TTL for each entry via
- * {@link #put(Object, Object, long, TimeUnit)} method.
+ * {@link RSetCache#add(Object, long, TimeUnit)} method.
  * </p>
  *
  * <p>Current Redis implementation doesn't have set entry eviction functionality.
@@ -51,11 +48,10 @@
  * In addition there is {@link org.redisson.EvictionScheduler}. This scheduler
  * deletes expired entries in time interval between 5 seconds to 2 hours.</p>
  *
- * <p>If eviction is not required then it's better to use {@link org.redisson.reactive.RedissonSet}.</p>
+ * <p>If eviction is not required then it's better to use {@link org.redisson.api.RSet}.</p>
  *
  * @author Nikita Koksharov
  *
- * @param <K> key
  * @param <V> value
  */
 public class RedissonSetCache<V> extends RedissonExpirable implements RSetCache<V> {

redisson/src/main/java/org/redisson/RedissonSortedSet.java

@@ -44,7 +44,7 @@
  *
  * @author Nikita Koksharov
  *
- * @param <V>
+ * @param <V> value
  */
 public class RedissonSortedSet<V> extends RedissonObject implements RSortedSet<V> {
 

redisson/src/main/java/org/redisson/api/RCollectionAsync.java

@@ -17,6 +17,12 @@
 
 import java.util.Collection;
 
+/**
+ * 
+ * @author Nikita Koksharov
+ *
+ * @param <V> type of value
+ */
 public interface RCollectionAsync<V> extends RExpirableAsync {
 
     /**

redisson/src/main/java/org/redisson/api/RCollectionReactive.java

@@ -19,6 +19,12 @@
 
 import org.reactivestreams.Publisher;
 
+/**
+ * 
+ * @author Nikita Koksharov
+ *
+ * @param <V> value
+ */
 public interface RCollectionReactive<V> extends RExpirableReactive {
 
     Publisher<V> iterator();

redisson/src/main/java/org/redisson/api/RHyperLogLog.java

@@ -17,6 +17,12 @@
 
 import java.util.Collection;
 
+/**
+ * 
+ * @author Nikita Koksharov
+ *
+ * @param <V> value
+ */
 public interface RHyperLogLog<V> extends RExpirable, RHyperLogLogAsync<V> {
 
     boolean add(V obj);

redisson/src/main/java/org/redisson/api/RHyperLogLogAsync.java

@@ -17,6 +17,12 @@
 
 import java.util.Collection;
 
+/**
+ * 
+ * @author Nikita Koksharov
+ *
+ * @param <V> value
+ */
 public interface RHyperLogLogAsync<V> extends RExpirableAsync {
 
     RFuture<Boolean> addAsync(V obj);

redisson/src/main/java/org/redisson/api/RSet.java

@@ -29,14 +29,14 @@ public interface RSet<V> extends Set<V>, RExpirable, RSetAsync<V> {
     /**
      * Removes and returns random element from set
      *
-     * @return
+     * @return value
      */
     V removeRandom();
 
     /**
      * Returns random element from set
      *
-     * @return
+     * @return value
      */
     V random();
 
@@ -53,61 +53,61 @@ public interface RSet<V> extends Set<V>, RExpirable, RSetAsync<V> {
     /**
      * Read all elements at once
      *
-     * @return
+     * @return values
      */
     Set<V> readAll();
 
     /**
      * Union sets specified by name and write to current set.
      * If current set already exists, it is overwritten.
      *
-     * @param names
-     * @return
+     * @param names - name of sets
+     * @return size of union
      */
     int union(String... names);
 
     /**
-     * Union sets specified by name with current set.
-     * Without current set state change.
-     *
-     * @param names
-     * @return
+     * Union sets specified by name with current set
+     * without current set state change.
+     * 
+     * @param names - name of sets
+     * @return values
      */
     Set<V> readUnion(String... names);
 
     /**
      * Diff sets specified by name and write to current set.
      * If current set already exists, it is overwritten.
      *
-     * @param names
-     * @return
+     * @param names - name of sets
+     * @return values
      */
     int diff(String... names);
 
     /**
      * Diff sets specified by name with current set.
      * Without current set state change.
-     *
-     * @param names
-     * @return
+     * 
+     * @param names - name of sets
+     * @return values
      */
 
     Set<V> readDiff(String... names);
     /**
      * Intersection sets specified by name and write to current set.
      * If current set already exists, it is overwritten.
      *
-     * @param names
-     * @return
+     * @param names - name of sets
+     * @return size of intersection
      */
     int intersection(String... names);
 
     /**
-     * Intersection sets specified by name with current set.
-     * Without current set state change.
-     *
-     * @param names
-     * @return
+     * Intersection sets specified by name with current set
+     * without current set state change.
+     * 
+     * @param names - name of sets
+     * @return values
      */
     Set<V> readIntersection(String... names);