spring-projects / spring-projects/spring-data-redis

Autodetect and init lua scripts. [DATAREDIS-281]

Open
#858 0 comments 0 reactions 1 assignee View on GitHub

@thomasdarimont is already working on this.

Since Dec 30, 2020.

in: core type: enhancement
Dominant language
Java
Stars
1.9k
Forks
1.3k
No merged PRs in 30d

Description

Christoph Strobl opened DATAREDIS-281 and commented

At this time lua sctipts have to be initialized by hand

@Bean
public RedisScrip<Boolean> checkAndSet() {
  DefaultRedisScript<Boolean> redisScript = new DefaultRedisScript<Boolean>();
  redisScript.setScriptSource(new ResourceScriptSource(new ClassPathResource("META-INF/scripts/checkAndSet.lua")));
  redisScript.setResultType(Boolean.class);
}

It would be nice to pick up *.lua files following a naming convention on return types by default.

eg. boolean-checkAndSet.lua could replace the sample code above

No further details from DATAREDIS-281

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.