Skip to content

Using official Golang Redis library emits "Command not found" when using HSet #201

Closed

Description

@scwagner

ArcadeDB Version: v21.11.1-SNAPSHOT (build 0de6fd8/1636358759681/main)

JDK Version: OpenJDK 64-Bit Server VM Temurin-11.0.13+8 (build 11.0.13+8, mixed mode)

OS: Alpine Linux v3.14.2

Expected behavior

The official Redis client for Golang should work with ArcadeDB

Actual behavior

Call HSet using the official library returns the error Command not found

Steps to reproduce

1. Run the version above of ArcadeDB in Docker with RedisProtocolPlugin enabled and ports forwarded

docker run --pull always -d --rm --name arcadedb -p 6379:6379 -p 2480:2480 -p 2424:2424 --env arcadedb.server.rootPassword=playwithdata --env arcadedb.server.plugins="Redis:com.arcadedb.redis.RedisProtocolPlugin" arcadedata/arcadedb:latest

2. Follow the instructions from the documentation for creating the sample document type in the database:

https://docs.arcadedb.com/#Redis-API
Browse down to the Persistent Commands and run the block of commands provided for creating the Account document type.

3. Build and run Go application (Gist included)

The following gist contains a minimum Go application to reproduce the issue:
https://gist.github.com/scwagner/5e4d6e015a2a13bb3c82df72733f0d07

With Go 1.17 installed, create a directory with the 3 files from that gist in it, and execute go run main.go.

Activity

  1. lvca commented on Nov 8, 2021

    @lvca
    Member

    @scwagner I was able to reproduce your issue by installing redis-cli. I noticed commands are sent lowercase, while with the official java redis driver they are all upper case. My fix is to have case insensitive commands now and now it works with redis-cli tool, so it should work with GoLang driver too. Please let me know if don't and in case please reopen the issue.

  2. lvca commented on Nov 8, 2021

    @lvca
    Member

    If you're using docker, please wait a little while the CI is building and publishing it.

  3. lvca commented on Nov 8, 2021

    @lvca
    Member

    Thanks @scwagner, please keep using and testing the Redis module!

  4. scwagner commented on Nov 9, 2021

    @scwagner
    Author

    Thanks @lvca I can confirm that the latest build on DockerHub is now working with the Go library.

  5. lvca commented on Nov 9, 2021

    @lvca
    Member

    Great, thanks for confirming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions