Skip to content

Add new MySQL scanner option to extension_entries#18759

Closed
staticlibs wants to merge 1 commit into
duckdb:mainfrom
staticlibs:mysql_time_option
Closed

Add new MySQL scanner option to extension_entries#18759
staticlibs wants to merge 1 commit into
duckdb:mainfrom
staticlibs:mysql_time_option

Conversation

@staticlibs
Copy link
Copy Markdown
Member

This PR adds new MySQL scanner option mysql_time_as_time (that is being added in duckdb/duckdb-mysql#142) to extension_entries.hpp.

It is intended to be merged after the duckdb/duckdb-mysql#142 is reviewed and merged.

This PR adds new MySQL scanner option `mysql_time_as_time` (that is
being added in duckdb/duckdb-mysql#142) to `extension_entries.hpp`.

It is intended to be merged after the duckdb/duckdb-mysql#142 is
reviewed and merged.
@Mytherin
Copy link
Copy Markdown
Collaborator

Mytherin commented Aug 27, 2025

Thanks for the PR!

This won't work as the extension option doesn't exist yet. In order to get this in you need to bump the MySQL hash that is referenced in the MySQL extension that is built (.github/config/extensions/mysql_scanner.cmake) to the hash that has the option, together with adding the option - but that would first require the PR MySQL side to be merged.

@staticlibs staticlibs marked this pull request as draft August 27, 2025 15:55
@staticlibs
Copy link
Copy Markdown
Member Author

Closing as superseded by #18848.

@staticlibs staticlibs closed this Sep 3, 2025
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