search

coinbase / chainstorage

The File System For a Multi-Blockchain World
https://time-wilderness-a70.notion.site/ChainStorage-5c173d1cafd842ceb9e38c26bfbd6265?pvs=4
Apache License 2.0
69 stars 24 forks source link

ChainStorage workflow ergonomic #34

Closed leozc closed 8 months ago

leozc commented 9 months ago

ChainStorage is reliant on Temporal, which is a crucial dependency. However, it currently operates on an outdated version of Temporal. Even the current Temporal UI Docker is no longer supported.

It would be advisable to modify ChainStorage so that it can operate on a readily available Temporal deployment.

bestmike007 commented 9 months ago

I've already upgraded the docker images for running temporal and the ui in https://github.com/coinbase/chainstorage/pull/29. It doesn't seem to have any compatibility issues.

However, there are breaking changes in temporal sdk, which might take some time.

jiezhang commented 9 months ago

@leozc We have upgraded the temporal dependencies in the internal repo. We need to find some time to sync the changes to the public one.

leozc commented 8 months ago

Sure I can drop this then