Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Eval operation not supported #12

Closed
vepo opened this issue Jan 4, 2019 · 7 comments · Fixed by atp-mipt/jedis-mock#155
Closed

Eval operation not supported #12

vepo opened this issue Jan 4, 2019 · 7 comments · Fixed by atp-mipt/jedis-mock#155
Labels
enhancement

Comments

@vepo
Copy link

vepo commented Jan 4, 2019

I'm using jedis-mock with Redisson. When I call bucket.getAndDelete(), the lib is throwing the following exception:

15:49:42.811 [Thread-41] ERROR c.g.f.j.c.RedisOperationExecutor - Malformed request
java.lang.UnsupportedOperationException: Unsupported operation 'eval'
	at com.github.fppt.jedismock.commands.RedisOperationExecutor.buildSimpleOperation(RedisOperationExecutor.java:148)
	at com.github.fppt.jedismock.commands.RedisOperationExecutor.execCommand(RedisOperationExecutor.java:166)
	at com.github.fppt.jedismock.RedisClient.run(RedisClient.java:48)
	at java.lang.Thread.run(Thread.java:748)

Is eval already supported?

How can I make it work?
@vepo vepo changed the title Operation not supported Eval operation not supported Jan 4, 2019
@fppt
Copy link
Owner

fppt commented Jan 9, 2019

eval is not yet supported. I will make sure to add it for the next release though. Thanks for raising the issue!

@fppt
Copy link
Owner

fppt commented Jan 28, 2019

So this is turning out more time consuming than I though. i started working with this: https://github.com/luaj/luaj but have not had any luck so far. May have to wait a few releases until I figure this one out. Unless someone can suggest a good way to get java to interpret lua script

@kazetsukaimiko
Copy link

@fppt What was the issue with luaj integration?

You probably not only need the lua interpreter support, but also to expose any redis functionality to lua that is a part of the redis api.

@mnsrmov
Copy link

mnsrmov commented Jun 23, 2020

I'm having the same issue on the latest version in maven repository. Is eval still not supported?

fppt pushed a commit that referenced this issue Dec 21, 2021
@inponomarev @pekashy
@julia-ste @aleksandr1101 @dependabot @gogagum
Merge v. 1.0.0 changes (#154)
39e86e5 
* Refactor expiring key value storage (#2)


Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Refactor operation factory (#4)

* refactor operation factory

* wait-notify for waiting operations

* update reflections, eager static initialization of CommandFactory

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* remove debug output

* guava deleted (#5)

* guava deleted

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* add dependabot (#6)

* remove Publish Test Report

it does not work well with dependabot's PRs

* Bump mockito-core from 3.10.0 to 4.0.0 (#15)

Bumps [mockito-core](https://github.com/mockito/mockito) from 3.10.0 to 4.0.0.
- [Release notes](https://github.com/mockito/mockito/releases)
- [Commits](mockito/mockito@v3.10.0...v4.0.0)

---
updated-dependencies:
- dependency-name: org.mockito:mockito-core
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump slf4j-api from 1.7.20 to 1.7.32 (#12)

Bumps [slf4j-api](https://github.com/qos-ch/slf4j) from 1.7.20 to 1.7.32.
- [Release notes](https://github.com/qos-ch/slf4j/releases)
- [Commits](qos-ch/slf4j@v_1.7.20...v_1.7.32)

---
updated-dependencies:
- dependency-name: org.slf4j:slf4j-api
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump junit-jupiter from 5.5.2 to 5.8.1 (#9)

Bumps [junit-jupiter](https://github.com/junit-team/junit5) from 5.5.2 to 5.8.1.
- [Release notes](https://github.com/junit-team/junit5/releases)
- [Commits](junit-team/junit5@r5.5.2...r5.8.1)

---
updated-dependencies:
- dependency-name: org.junit.jupiter:junit-jupiter
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump maven-source-plugin from 2.2.1 to 3.2.1 (#14)

Bumps [maven-source-plugin](https://github.com/apache/maven-source-plugin) from 2.2.1 to 3.2.1.
- [Release notes](https://github.com/apache/maven-source-plugin/releases)
- [Commits](apache/maven-source-plugin@maven-source-plugin-2.2.1...maven-source-plugin-3.2.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-source-plugin
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump maven-javadoc-plugin from 2.9.1 to 3.3.1 (#13)

Bumps [maven-javadoc-plugin](https://github.com/apache/maven-javadoc-plugin) from 2.9.1 to 3.3.1.
- [Release notes](https://github.com/apache/maven-javadoc-plugin/releases)
- [Commits](apache/maven-javadoc-plugin@maven-javadoc-plugin-2.9.1...maven-javadoc-plugin-3.3.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-javadoc-plugin
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump maven-compiler-plugin from 3.1 to 3.8.1 (#10)

Bumps [maven-compiler-plugin](https://github.com/apache/maven-compiler-plugin) from 3.1 to 3.8.1.
- [Release notes](https://github.com/apache/maven-compiler-plugin/releases)
- [Commits](apache/maven-compiler-plugin@maven-compiler-plugin-3.1...maven-compiler-plugin-3.8.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-compiler-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump checkstyle from 8.42 to 9.1 (#11)

Bumps [checkstyle](https://github.com/checkstyle/checkstyle) from 8.42 to 9.1.
- [Release notes](https://github.com/checkstyle/checkstyle/releases)
- [Commits](checkstyle/checkstyle@checkstyle-8.42...checkstyle-9.1)

---
updated-dependencies:
- dependency-name: com.puppycrawl.tools:checkstyle
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump maven-gpg-plugin from 1.5 to 3.0.1 (#16)

Bumps [maven-gpg-plugin](https://github.com/apache/maven-gpg-plugin) from 1.5 to 3.0.1.
- [Release notes](https://github.com/apache/maven-gpg-plugin/releases)
- [Commits](apache/maven-gpg-plugin@maven-gpg-plugin-1.5...maven-gpg-plugin-3.0.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-gpg-plugin
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump maven-pmd-plugin from 3.7 to 3.15.0 (#8)

Bumps [maven-pmd-plugin](https://github.com/apache/maven-pmd-plugin) from 3.7 to 3.15.0.
- [Release notes](https://github.com/apache/maven-pmd-plugin/releases)
- [Commits](apache/maven-pmd-plugin@maven-pmd-plugin-3.7...maven-pmd-plugin-3.15.0)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-pmd-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump lettuce-core from 6.1.1.RELEASE to 6.1.5.RELEASE (#18)

Bumps [lettuce-core](https://github.com/lettuce-io/lettuce-core) from 6.1.1.RELEASE to 6.1.5.RELEASE.
- [Release notes](https://github.com/lettuce-io/lettuce-core/releases)
- [Changelog](https://github.com/lettuce-io/lettuce-core/blob/6.1.5.RELEASE/RELEASE-NOTES.md)
- [Commits](redis/lettuce@6.1.1.RELEASE...6.1.5.RELEASE)

---
updated-dependencies:
- dependency-name: io.lettuce:lettuce-core
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump jedis from 3.6.0 to 3.7.0 (#17)

Bumps [jedis](https://github.com/redis/jedis) from 3.6.0 to 3.7.0.
- [Release notes](https://github.com/redis/jedis/releases)
- [Commits](redis/jedis@jedis-3.6.0...v3.7.0)

---
updated-dependencies:
- dependency-name: redis.clients:jedis
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump spotbugs version (#19)

* fix pubsub (#20)

* fix pubsub

* Remove slice serialization functions (#21)


* Store RMSet directly for pf and s operations

* Use RMList directly instead of Slice in l-operations

* Use RMHMap instead of Slice for z operations

* Remove not needed (de)serialisation code. Clean related imports

* Move some type-related logic from Expiring Key-Value store

* Remove wildcard import to match checkstyle

* Remove serializable interface from Slice

* Add polimorfic getTypeName to RMDataStructure children

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Bump maven-release-plugin from 2.5 to 2.5.3 (#22)

Bumps maven-release-plugin from 2.5 to 2.5.3.

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-release-plugin
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Supported operations generator (#23)

* Add markdown generator of supported operations

* Add generated list

* Update link to generated list

* Add GitHub Actions step to update supported operations list

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* use git-auto-commit action

* fix Test annotation

* Update list of supported operations

* add LINE_SEPARATOR

* Update list of supported operations

* set linebreaks

* Update list of supported operations

* Bump mockito-core from 4.0.0 to 4.1.0 (#24)

Bumps [mockito-core](https://github.com/mockito/mockito) from 4.0.0 to 4.1.0.
- [Release notes](https://github.com/mockito/mockito/releases)
- [Commits](mockito/mockito@v4.0.0...v4.1.0)

---
updated-dependencies:
- dependency-name: org.mockito:mockito-core
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* update redis container version (#27)

* Update list of supported operations

* Organize operations (#28)

* Move operations to packages

* Add operation categories

* Update structure of supported operations list

* Update list of supported operations

* add persist support (#53) (#31)

* add persist support

* rebase, move class & fix test

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Bump checkstyle from 9.1 to 9.2 (#30)

Bumps [checkstyle](https://github.com/checkstyle/checkstyle) from 9.1 to 9.2.
- [Release notes](https://github.com/checkstyle/checkstyle/releases)
- [Commits](checkstyle/checkstyle@checkstyle-9.1...checkstyle-9.2)

---
updated-dependencies:
- dependency-name: com.puppycrawl.tools:checkstyle
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump junit-jupiter from 5.8.1 to 5.8.2 (#29)

Bumps [junit-jupiter](https://github.com/junit-team/junit5) from 5.8.1 to 5.8.2.
- [Release notes](https://github.com/junit-team/junit5/releases)
- [Commits](junit-team/junit5@r5.8.1...r5.8.2)

---
updated-dependencies:
- dependency-name: org.junit.jupiter:junit-jupiter
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update list of supported operations

* Bump spotbugs-maven-plugin from 4.4.2.2 to 4.5.0.0 (#25)

* Bump spotbugs-maven-plugin from 4.4.2.2 to 4.5.0.0

Bumps [spotbugs-maven-plugin](https://github.com/spotbugs/spotbugs-maven-plugin) from 4.4.2.2 to 4.5.0.0.
- [Release notes](https://github.com/spotbugs/spotbugs-maven-plugin/releases)
- [Commits](spotbugs/spotbugs-maven-plugin@spotbugs-maven-plugin-4.4.2.2...spotbugs-maven-plugin-4.5.0.0)

---
updated-dependencies:
- dependency-name: com.github.spotbugs:spotbugs-maven-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* make spotbugs happy

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Update list of supported operations

* Organize tests (#33)

* Move tests to packages

* Remove ANSI escape code from output

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Update list of supported operations

* Mocked operations handler (#32)

* Added handler field and new constructors to classes.

* MockExecutor & RedisCommandInterceptor interface.

* refactor and update README.md

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Watch unwatch (#34)

* watch/unwatch commands added

* unwatch added to discard

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Update list of supported operations

* Add psubscribe, punsubscribe commands, pubSumNumPat subcommand (#35)

* Add psubscribe, punsubscribe commands, pubSumNumPat subcommand

Co-authored-by: Ivan Ponomarev <iponomarev@mail.ru>

* Update list of supported operations

* Bump jedis from 3.7.0 to 3.7.1 (#36)

Bumps [jedis](https://github.com/redis/jedis) from 3.7.0 to 3.7.1.
- [Release notes](https://github.com/redis/jedis/releases)
- [Commits](redis/jedis@v3.7.0...v3.7.1)

---
updated-dependencies:
- dependency-name: redis.clients:jedis
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump mockito-core from 4.1.0 to 4.2.0 (#37)

Bumps [mockito-core](https://github.com/mockito/mockito) from 4.1.0 to 4.2.0.
- [Release notes](https://github.com/mockito/mockito/releases)
- [Commits](mockito/mockito@v4.1.0...v4.2.0)

---
updated-dependencies:
- dependency-name: org.mockito:mockito-core
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* extra test based on #155 report

* Update README.md

Co-authored-by: Egor Zyazev <klever.kl@yandex.ru>
Co-authored-by: Julia Stepanova <yuliya.stepanova@phystech.edu>
Co-authored-by: Aleksandr Muratidi <60786024+aleksandr1101@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: inponomarev <inponomarev@users.noreply.github.com>
Co-authored-by: Georgy Guminov <43472625+gogagum@users.noreply.github.com>
@Simeon-Yang
Copy link

I'm having the same issue on the latest version "1.0.5" in maven repository. Is eval still not supported?

@inponomarev inponomarev mentioned this issue May 10, 2023
Merged
@inponomarev
Copy link
Collaborator

@vepo @kazetsukaimiko @mnsrmov @yang-yi-96 great news, after 4 years since the first issue the Lua scripting with LuaJ is finally implemented

@Simeon-Yang
Copy link

thank you very much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add_eval atp-mipt/jedis-mock
6 participants
@vepo @kazetsukaimiko @fppt @inponomarev @mnsrmov @Simeon-Yang