Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

feat(sqlite): move from warp-contracts-lmdb to warp-contracts-sqlite #60

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

dtfiedler
Copy link
Collaborator

@dtfiedler dtfiedler commented Nov 20, 2023

Were seeing state corruption in lmdb implementation. An issue was created with warp-contracts-lmdb, but for now we will migrate to sqlite.

Related: warp-contracts/warp-contracts-lmdb#26

Example log:

cacheKey":"fbU8Y4NMKKzP4rmAYeYj6tDrVDo9XNbdyq5IZPA31WQ-priceForInteraction-dbf5a69f55ab199c955d1c4df17aaf990435ea2b65e17c5aaa08af851b6dea27-7be80150a81463abc033a504e0ba48cd4ea8f93bdecb29b3e0d096ed09268237","contractTxId":"fbU8Y4NMKKzP4rmAYeYj6tDrVDo9XNbdyq5IZPA31WQ","errorMessage":"Error: Database is not open\n    at maybeOpened (/usr/src/app/node_modules/abstract-level/abstract-level.js:133:18)\n    at /usr/src/app/node_modules/abstract-level/abstract-level.js:160:13\n    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)","functionName":"priceForInteraction","input":{"interactionName":"increaseUndernameCount","name":"hoogerypokery","qty":278},"level":"error","message":"Read interaction failed!","method":"GET","path":"/v1/contract/fbU8Y4NMKKzP4rmAYeYj6tDrVDo9XNbdyq5IZPA31WQ/read/priceForInteraction","timestamp":"2023-11-20T22:24:40.671Z","trace":"6d2304"}
2023-11-20 16:24:40	
    error: 'Error: Database is not open\n' +
2023-11-20 16:24:40	
2023-11-20T22:24:40.670Z FATAL [HandlerBasedContract] Error while interacting with contract [
2023-11-20 16:24:40	
{"error":"Error: Database is not open\n    at maybeOpened (/usr/src/app/node_modules/abstract-level/abstract-level.js:133:18)\n    at /usr/src/app/node_modules/abstract-level/abstract-level.js:160:13\n    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)","level":"error","message":"Error processing request.","method":"GET","path":"/v1/contract/fbU8Y4NMKKzP4rmAYeYj6tDrVDo9XNbdyq5IZPA31WQ/read/priceForInteraction","timestamp":"2023-11-20T22:24:40.669Z","trace":"7f17b9"}
2023-11-20 16:24:40

Were seeing state corruption in lmdb implementation. An issue was created with warp-contracts-lmdb, but for now we will migrate to sqlite.
@dtfiedler dtfiedler requested a review from a team as a code owner November 20, 2023 18:30
@dtfiedler dtfiedler changed the title feat(sqlite): move fro warp-contracts-lmdb to warp-conracts-sqlite feat(sqlite): move from warp-contracts-lmdb to warp-conracts-sqlite Nov 20, 2023
@dtfiedler dtfiedler changed the title feat(sqlite): move from warp-contracts-lmdb to warp-conracts-sqlite feat(sqlite): move from warp-contracts-lmdb to warp-contracts-sqlite Nov 20, 2023
@dtfiedler
Copy link
Collaborator Author

we may not need this - holding off for now until we can confirm LMDB is not capable of handling our increasing workload

@dtfiedler dtfiedler merged commit f84c316 into main Nov 21, 2023
3 checks passed
@dtfiedler dtfiedler deleted the sqlite branch November 21, 2023 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants