Skip to content

Additional types for Firebird#190

Open
kgaughan wants to merge 1 commit intoMaxteabag:mainfrom
kgaughan:missing-types-firebird
Open

Additional types for Firebird#190
kgaughan wants to merge 1 commit intoMaxteabag:mainfrom
kgaughan:missing-types-firebird

Conversation

@kgaughan
Copy link
Copy Markdown
Contributor

@kgaughan kgaughan commented May 2, 2026

I'd missed these in my original PR. Additionally, this marks blobs of the text subtype as such and as there's the possibility of additional types being added in future that the adapter doesn't know about, it now prints 'UNKNOWN' for unknown types.

I'm working on getting the timezone test working properly, but currently it's crashing with an assertion error from within the driver. I'll open a separate PR for those. Unfortunately, passing timezone_datetime_type="TIMESTAMP WITH TIME ZONE" to the test configuration class's constructor appears not to be enough.

I'd missed these in my original PR. Additionally, this marks blobs of
the text subtype as such and as there's the possibility of additional
types being added in future that the adapter doesn't know about, it now
prints 'UNKNOWN' for unknown types.

I'm working on getting the timezone test working properly, but currently
it's crashing with an assertion error from within the driver.
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.

1 participant