Skip to content

curl client gets "Connection refused" error when rkv uses the default mem database backend #65

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.

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

Open
h-w-chen opened this issue Sep 1, 2022 · 4 comments

Comments

@h-w-chen
Copy link
Collaborator

h-w-chen commented Sep 1, 2022

get the latest version of rkv source code (commit de9428e), build and start rkv server; seemingly fine; however trying to curl curl http://127.0.0.1:8090/kv?key=a gets back unexpected response curl: (7) Failed to connect to 127.0.0.1 port 8090: Connection refused

The expected response is rev not found, since there is no such key yet

This seems a regression. Reverting back to prior commit, the curl gets response mvcc: Revision not found

@jshaofuturewei
Copy link
Collaborator

The default database store type has been changed to mem while memory databases are used for unit tests.

Please use redis databases when you start rkv locally.

@kxu1985
Copy link
Collaborator

kxu1985 commented Sep 14, 2022

I am also seeing this issue for consistency validation (sequential). Redis was deployed as backend. It affects the returned value (logged as 'nil' for some operations). Not sure if we have a workaround for this.

Btw, for consistency validation (linearizability) I did not see this issue (although used an older code version). But I am not sure if this is caused by sync/async configuration or recent code update.

Screen Shot 2022-09-14 at 4 51 02 PM

@jshaofuturewei
Copy link
Collaborator

The error in the log are the jaeger exceptions. It looks like there is no jaeger instance in the localhost

Can you provide the revision number along with the value(213)? Did it read from async node?

For the line 69 and 70, why the read values are different?

@kxu1985
Copy link
Collaborator

kxu1985 commented Sep 15, 2022

Thanks for the comments. I did forget attaching the jaeger parameter when relaunching the process after build. Now these errors are gone!!

For concurrent read with multiple async nodes, I am seeing some interesting behaviors. I will share more details in tomorrow's scrum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants