Skip to content

fix: distinguish enum from spatial column - #396

Merged
rafiss merged 1 commit into
masterfrom
fix/enum-name-conflict
Jan 21, 2026
Merged

fix: distinguish enum from spatial column#396
rafiss merged 1 commit into
masterfrom
fix/enum-name-conflict

Conversation

@BuonOmo

@BuonOmo BuonOmo commented Jan 8, 2026

Copy link
Copy Markdown
Collaborator

If an enum name contained a spatial type
name (e.g. "point"). It would be matched
by a regex as a sql type. This commit is
handling that by checking type metadata.

See rgeo/activerecord-postgis-adapter#432

@BuonOmo
BuonOmo force-pushed the fix/enum-name-conflict branch 2 times, most recently from a157e21 to 1784230 Compare January 20, 2026 09:55
@BuonOmo
BuonOmo marked this pull request as ready for review January 20, 2026 10:27

@rafiss rafiss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

great catch!

If an enum name contained a spatial type
name (e.g. "point"). It would be matched
by a regex as a sql type. This commit is
handling that by checking type metadata.

See rgeo/activerecord-postgis-adapter#432
@rafiss
rafiss force-pushed the fix/enum-name-conflict branch from 1784230 to cc3209c Compare January 20, 2026 17:12
@rafiss
rafiss merged commit f50f581 into master Jan 21, 2026
8 of 10 checks passed
@BuonOmo
BuonOmo deleted the fix/enum-name-conflict branch January 21, 2026 07:14
@rafiss rafiss mentioned this pull request Aug 19, 2026
rafiss added a commit that referenced this pull request Aug 19, 2026
- Disabled schema_locked by default on connect for CockroachDB v25.3+ and
  unlocked tables around batched foreign key changes (#404)
- Improved error classification by detecting cached plan failures from the
  error message instead of the source function (#403)
- Stopped advertising support for restarting database transactions (#398)
- Fixed enum columns being misdetected as spatial columns (#396)

Co-Authored-By: roachdev-claude <roachdev-claude-bot@cockroachlabs.com>
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.

2 participants