Skip to content

Conversation

@codeaucafe
Copy link
Contributor

Summary

Add adapter and test expectations for the new dolt_status_ignored system table from dolthub/dolt#10227.

This table provides the same information as dolt_status plus an additional ignored boolean column.

Changes

  • Add DoltgresDoltStatusIgnoredTableAdapter in server/tables/dtables/status_ignored.go
  • Register adapter in server/tables/dtables/init.go
  • Add new table to test expectations in:
    • testing/go/pgcatalog_test.go
    • testing/go/sequences_test.go

Dependencies

Add adapter and test expectations for the new dolt_status_ignored
system table from PR dolthub/dolt#10227. This table provides the same
information as dolt_status plus an additional ignored column
indicating whether unstaged new tables match a pattern in dolt_ignore.

Changes:
- Add DoltgresDoltStatusIgnoredTableAdapter in status_ignored.go
- Register DoltgresDoltStatusIgnoredTableAdapter in init.go
- Update pgcatalog_test.go with new dolt_status_ignore table to expected
tables
- Add new dolt_status_ignore table to sequences_test.go expected tables

Refs: dolthub/dolt#5862, dolthub/dolt#10227
Due to codeaucafe's contribution being on a fork updating the
doltgresql's go.mod to point to the updates in the dolt PR,
dolthub/dolt#10227 this requires a replace directive. This will at least
allow us to ensure this DoltgreSQL's PR can run CI.

Refs: dolthub/dolt#5862, dolthub/dolt#10227
Update pg_catalog test expectations to include new status_ignored system
table in the dolt schema.

Refs: dolthub/dolt#5862, dolthub/dolt#10227
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants