Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite C# server modules off V8 to V9 #1948

Closed
RReverser opened this issue Nov 5, 2024 · 2 comments · Fixed by #1670
Closed

Rewrite C# server modules off V8 to V9 #1948

RReverser opened this issue Nov 5, 2024 · 2 comments · Fixed by #1670
Assignees

Comments

@RReverser
Copy link
Member

No description provided.

@RReverser RReverser self-assigned this Nov 5, 2024
@RReverser
Copy link
Member Author

RReverser commented Nov 5, 2024

In-progress PR in #1670. It's currently blocked on final changes in #1918.

@bfops
Copy link
Collaborator

bfops commented Nov 22, 2024

We plan to remove module def v8 by 1.0

RReverser added a commit that referenced this issue Nov 28, 2024
Migrated C# ModuleDef to V9 as well as adds tests to ensure that identical modules indeed produce identical schemas.

The two remaining discrepancies between Rust and C# are:
 - #1891
 - Row-level security not having a designed API & implementation for C#.

 - Fixes #1948.
 - Fixes #1589.
 - Supersedes #1619.
RReverser added a commit that referenced this issue Nov 28, 2024
Migrated C# ModuleDef to V9 as well as adds tests to ensure that identical modules indeed produce identical schemas.

The two remaining discrepancies between Rust and C# are:
 - #1891
 - Row-level security not having a designed API & implementation for C#.

 - Fixes #1948.
 - Fixes #1589.
 - Supersedes #1619.
RReverser added a commit that referenced this issue Dec 9, 2024
Migrated C# ModuleDef to V9 as well as adds tests to ensure that identical modules indeed produce identical schemas.

The two remaining discrepancies between Rust and C# are:
 - #1891
 - Row-level security not having a designed API & implementation for C#.

 - Fixes #1948.
 - Fixes #1589.
 - Supersedes #1619.
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 a pull request may close this issue.

2 participants