weavedb / rdk

WeaveDB Rollup Development Kit
3 stars 3 forks source link

add test case for transactions not evaluated on weavedb-sdk-node with nocache set to false #24

Closed drumfeet closed 6 months ago

drumfeet commented 7 months ago

I think it is still okay to merge the PR even if this turns out not to be an issue. The process must remain active until the SDK successfully pushes the transaction to Warp.

ocrybit commented 6 months ago

This is indeed correct behavior. You need to make sure the query is finalized with await tx.getResult() to make it consistent with another SDK instance. Or you can use the same instance to query the data after the update.