diff --git a/src/main/java/org/springframework/data/redis/core/script/DefaultRedisScript.java b/src/main/java/org/springframework/data/redis/core/script/DefaultRedisScript.java index c78a26a29..54a373803 100644 --- a/src/main/java/org/springframework/data/redis/core/script/DefaultRedisScript.java +++ b/src/main/java/org/springframework/data/redis/core/script/DefaultRedisScript.java @@ -46,8 +46,7 @@ public class DefaultRedisScript implements RedisScript, InitializingBean { /** * Creates a new {@link DefaultRedisScript} */ - public DefaultRedisScript() { - } + public DefaultRedisScript() {} /** * Creates a new {@link DefaultRedisScript} @@ -117,9 +116,9 @@ public class DefaultRedisScript implements RedisScript, InitializingBean { /** * @param resultType The script result type. Should be one of Long, Boolean, List, or deserialized value type. Can be - * null if the script returns a throw-away status (i.e "OK") + * {@literal null} if the script returns a throw-away status (i.e "OK") */ - public void setResultType(Class resultType) { + public void setResultType(@Nullable Class resultType) { this.resultType = resultType; } diff --git a/src/main/java/org/springframework/data/redis/core/script/RedisScript.java b/src/main/java/org/springframework/data/redis/core/script/RedisScript.java index 67c16e204..d478b0814 100644 --- a/src/main/java/org/springframework/data/redis/core/script/RedisScript.java +++ b/src/main/java/org/springframework/data/redis/core/script/RedisScript.java @@ -84,10 +84,11 @@ public interface RedisScript { } /** - * Creates new {@link RedisScript} from {@link Resource}. + * Creates new {@link RedisScript} (with throw away result) from the given {@link Resource}. * * @param resource must not be {@literal null}. * @return new instance of {@link RedisScript}. + * @throws IllegalArgumentException if the required argument is {@literal null}. * @since 2.2 */ static RedisScript of(Resource resource) { @@ -96,7 +97,6 @@ public interface RedisScript { DefaultRedisScript script = new DefaultRedisScript<>(); script.setLocation(resource); - script.afterPropertiesSet(); return script; } @@ -107,6 +107,7 @@ public interface RedisScript { * @param resource must not be {@literal null}. * @param resultType must not be {@literal null}. * @return new instance of {@link RedisScript}. + * @throws IllegalArgumentException if any required argument is {@literal null}. * @since 2.2 */ static RedisScript of(Resource resource, Class resultType) { @@ -117,7 +118,6 @@ public interface RedisScript { DefaultRedisScript script = new DefaultRedisScript<>(); script.setResultType(resultType); script.setLocation(resource); - script.afterPropertiesSet(); return script; }