Skip to content

Conversation

@md0x
Copy link
Contributor

@md0x md0x commented Jan 15, 2026

Fixes UMA-3008

Subgraph fails at block 81660211 with error: index row size 3096 exceeds btree version 4 maximum 2704. Long ancillaryData from Polymarket resolution criteria causes OptimisticPriceRequest entity IDs to exceed PostgreSQL's btree index limit.

Solution: Changed the ID construction to hash the ancillaryData using keccak256, producing a fixed-length 66-character hash.

@linear
Copy link

linear bot commented Jan 15, 2026

@md0x md0x requested a review from gsteenkamp89 January 15, 2026 10:34
Copy link
Contributor

@gsteenkamp89 gsteenkamp89 left a comment

Choose a reason for hiding this comment

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

This looks good.
My only concern is that if any of these get disputed, they will move into the voting contract, so the voting subgraph will suffer from the same problem.

We should make similar changes to the voting subgraph's ID generation as well.
While we're at it, we can do the same for the OOv2 subgraph.

@gsteenkamp89
Copy link
Contributor

gsteenkamp89 commented Jan 15, 2026

I have verified that

  1. Nowhere in the oracle dapp do we attempt to query a single entity by constructing it's ID ✅
  2. The Voting dapp does do this, so we would have to ensure we mimic the logic here to construct the ID. ❌

md0x added 2 commits January 15, 2026 12:58
Signed-off-by: Pablo Maldonado <pablo@umaproject.org>
Copy link
Contributor

@gsteenkamp89 gsteenkamp89 left a comment

Choose a reason for hiding this comment

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

Looks good!
Thanks for getting this in.

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.

3 participants