Skip to content

Extension Types#5

Open
connortsui20 wants to merge 4 commits intodevelopfrom
ct/ext-types
Open

Extension Types#5
connortsui20 wants to merge 4 commits intodevelopfrom
ct/ext-types

Conversation

@connortsui20
Copy link
Contributor

@connortsui20 connortsui20 commented Feb 23, 2026

Rendered

This RFC proposes extending the ExtVTable trait to support richer behavior for extension types beyond forwarding to the storage type.

It covers the already completed vtable infrastructure, the proposed trait design for types/scalars/arrays, and identifies open questions around the extension array API and compute dispatch.

The big open question here is if we need to do this at all. And if we do need to do this, what should the extension array API look like? Importantly, how should we define expressions over extension type arrays?

@a10y
Copy link
Contributor

a10y commented Feb 23, 2026

You might want to rebase, and then you can get local previews using bun run dev

@connortsui20 connortsui20 force-pushed the ct/template branch 2 times, most recently from d0ece9c to d4cb849 Compare February 25, 2026 18:21
Base automatically changed from ct/template to develop February 25, 2026 18:22
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@connortsui20 connortsui20 marked this pull request as ready for review February 27, 2026 18:39
@connortsui20
Copy link
Contributor Author

connortsui20 commented Feb 27, 2026

I noticed that both footnotes and syntax highlighting seem to not be supported.

Edit: Syntax highlighting PR #17

Copy link
Contributor

@gatesn gatesn left a comment

Choose a reason for hiding this comment

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

I'm not sure what this RFC is proposing...

I think what is being proposed already exists on develop, and the unresolved questions remain unresolved?

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.

3 participants