IntersectMBO / cardano-db-sync

A component that follows the Cardano chain and stores blocks and transactions in PostgreSQL
Apache License 2.0
290 stars 162 forks source link

Fix committee and indexes #1737

Closed kderme closed 3 months ago

kderme commented 3 months ago

Description

Fixes https://github.com/IntersectMBO/cardano-db-sync/issues/1735. Also fixes the indexes It also cherry-picks some forgotten commits from release 13.2.0.2

Checklist

Migrations

If there is a breaking change, especially a big one, please add a justification here. Please elaborate more what the migration achieves, what it cannot achieve or why a migration is not possible.

sgillespie commented 3 months ago

Looks like you have some tests to update:

force-indexes
        check force-index adds indexes:                                          FAIL (51.21s)
          test/Test/Cardano/Db/Mock/Validate.hs:153:
          unexpected number of indexes: got 171 expected 172
          Use -p '/Conway unit tests.Command Line Arguments.force-indexes.check force-index adds indexes/' to rerun this test only.
        check no force-index doesn't add indexes:                                FAIL (51.22s)
          test/Test/Cardano/Db/Mock/Validate.hs:153:
          unexpected number of indexes: got 108 expected 107
          Use -p '/Conway unit tests.Command Line Arguments.force-indexes.check no force-index doesn'\''t add indexes/' to rerun this test only.
kderme commented 3 months ago

Done at https://github.com/IntersectMBO/cardano-db-sync/pull/1744