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.

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

parachain/disputes: dispute coordinator backend #3344

Merged
merged 12 commits into from
Sep 18, 2023

Conversation

kanishkatn
Copy link
Contributor

@kanishkatn kanishkatn commented Jun 19, 2023

@kanishkatn kanishkatn force-pushed the feat/k/dispute-backend branch 2 times, most recently from 7591e26 to e8a5787 Compare June 20, 2023 15:05
@kanishkatn kanishkatn changed the base branch from feat/k/dispute-coordinator to feat/k/participation-queue June 21, 2023 13:33
@kanishkatn kanishkatn changed the base branch from feat/k/participation-queue to feat/k/dispute-participation June 21, 2023 13:34
@kanishkatn kanishkatn force-pushed the feat/k/dispute-participation branch from 6b71e22 to 178c903 Compare June 21, 2023 13:45
@kanishkatn kanishkatn force-pushed the feat/k/dispute-backend branch 2 times, most recently from 985cf97 to cb3c834 Compare June 21, 2023 14:39
@kanishkatn kanishkatn force-pushed the feat/k/dispute-participation branch 6 times, most recently from 9dc5a20 to 7cab038 Compare June 23, 2023 12:08
@kanishkatn kanishkatn force-pushed the feat/k/dispute-backend branch 5 times, most recently from e4e82d7 to 599c163 Compare June 26, 2023 13:33
@kanishkatn kanishkatn force-pushed the feat/k/dispute-participation branch 6 times, most recently from 593d728 to ace8c36 Compare July 11, 2023 08:40
@kanishkatn kanishkatn force-pushed the feat/k/dispute-backend branch 3 times, most recently from 83d8580 to 65dfe4c Compare July 11, 2023 20:02
@kanishkatn kanishkatn force-pushed the feat/k/dispute-backend branch 2 times, most recently from 1df6fa6 to 5bb1f74 Compare August 23, 2023 17:53
dot/parachain/dispute/backend.go Outdated Show resolved Hide resolved
dot/parachain/dispute/backend.go Outdated Show resolved Hide resolved
dot/parachain/dispute/backend.go Outdated Show resolved Hide resolved
@EclesioMeloJunior
Copy link
Member

It looks like pebbleDb doesn't support transactions. It would require a bit of effort to implement the interface. Here's where it is mostly used https://github.com/paritytech/polkadot/blob/master/node/core/dispute-coordinator/src/db/v1.rs#L139
Do you think it would be an easy switch or could we continue to use badger in the subsystem?

You're right, but AFAIK you can use the pebble's Batcher which could replace the current badger transactions, more exactly the IndexedBatcher (https://github.com/cockroachdb/pebble/blob/master/batch.go), basically you can aggregate operations and commit or reset them

@kanishkatn
Copy link
Contributor Author

It looks like pebbleDb doesn't support transactions. It would require a bit of effort to implement the interface. Here's where it is mostly used https://github.com/paritytech/polkadot/blob/master/node/core/dispute-coordinator/src/db/v1.rs#L139
Do you think it would be an easy switch or could we continue to use badger in the subsystem?

You're right, but AFAIK you can use the pebble's Batcher which could replace the current badger transactions, more exactly the IndexedBatcher (https://github.com/cockroachdb/pebble/blob/master/batch.go), basically you can aggregate operations and commit or reset them

hmm, that's interesting. I had come across this old issue cockroachdb/pebble#581. In their readme, they suggest rocksDB for transactions as well. I'm gonna have to go through it, but for now I'd like to continue with the integration tests. How about we revisit this at a later stage?

Copy link
Contributor

@jimjbrettj jimjbrettj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some remaining comments, but lgtm

Base automatically changed from feat/k/dispute-participation to feat/k/disputes-coordinator September 18, 2023 12:18
@kanishkatn kanishkatn merged commit 232f34e into feat/k/disputes-coordinator Sep 18, 2023
17 of 21 checks passed
@kanishkatn kanishkatn deleted the feat/k/dispute-backend branch September 18, 2023 12:32
@kanishkatn
Copy link
Contributor Author

We still need to do the pebbleDB implementation. I'll pick it up in the end.

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

Successfully merging this pull request may close these issues.

4 participants