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

Last batch card in Barn is updated with a delay if place an order in Prod API #139

Closed
elena-zh opened this issue Jun 23, 2022 · 3 comments
Closed
Labels
app:Explorer Explorer App Enhancement New feature or request Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@elena-zh
Copy link

elena-zh commented Jun 23, 2022

I have noticed, that Last batch card in Barn (and all the rest cards as well) are updated with a dellay if place an order in Prod, but see the stats in Barn home page.

Delay might take:

  • for GC up to 6 minutes (TX 0xf058723cfa2935d5dd8a3fe7a73fa48dbd2635a87dbe124dc57602ccaadfd913)
  • for Mainnet - up tp 3

As a result, the last batch ID might show irrelevant info

Expected: reduce a delay?

I have to notice, that executed orders in BARN appear immediately on cards

@elena-zh elena-zh added app:Explorer Explorer App Enhancement New feature or request Low Severity indicator for defects. It won't cause any major break-down of the system Protofire labels Jun 23, 2022
@ramirotw
Copy link
Contributor

@elena-zh could you run some queries to the subgraphs after an order is confirmed to debug this further?

query LastBatch {
	settlements(orderBy: firstTradeTimestamp, orderDirection: desc, first: 1) {
    txHash
    firstTradeTimestamp
  }
	_meta {
		block {
			hash
			number
		}
	}
}

Mainnet subgraph: https://api.thegraph.com/subgraphs/name/cowprotocol/cow/graphql
GC subgraph: https://api.thegraph.com/subgraphs/name/cowprotocol/cow-gc/graphql

The txHash in the query should be updated accordingly.

@elena-zh
Copy link
Author

elena-zh commented Jun 28, 2022

Hey @ramirotw , I have retested the issue today.
I have to mention, that the issue is no longer relevant in GC, as the data was updated immediately there.

As for the Mainnet, the data was updated in 1 minute. I did not record the whole process, but here is the video where you can see that updated batchID appears with timing '1 minute ago':
https://watch.screencastify.com/v/F2Qm5bU50KdUSrMjlcrY

I'm not sure if this timing is acceptable. If yes, lets close this issue.

@elena-zh
Copy link
Author

elena-zh commented Jul 7, 2022

Closing the issue as I'm no longer able to reproduce it.

@elena-zh elena-zh closed this as completed Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
app:Explorer Explorer App Enhancement New feature or request Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Development

No branches or pull requests

2 participants