Skip to content

Conversation

sanjeev23oct
Copy link

Postgres Function support added

Why make this change?

  • Reference associated issue using # syntax. e.g. Closes #XX
    • Include summary (1-2 sentences) of linked issue to avoid redirecting reviewers to different pages.
    • Include links to any additional discussion threads related to this change.

What is this change?

  • Summary of how your changes work to give reviewers context of your intent.
    • Links to relevant documentation / StackOverflow, if applicable.

How was this tested?

  • Integration Tests
  • Unit Tests

Sample Request(s)

  • Example REST and/or GraphQL request to demonstrate modifications
  • Example of CLI usage to demonstrate modifications

Postgres Function support added
@sanjeev23oct
Copy link
Author

@microsoft-github-policy-service agree

@seantleonard
Copy link
Contributor

Thank you for your contribution , @sanjeev23oct. Can you fill out the PR description template with contextual info? Please also add tests for validating your changes, which will help us review.

/// <summary>
/// Takes a string version of a MySql data type and returns its .NET common language runtime (CLR) counterpart
/// TODO: For MySql stored procedure support, this needs to be implemented.
/// Takes a string version of a PostgreSql data type and returns its .NET common language runtime (CLR) counterpart
Copy link
Contributor

Choose a reason for hiding this comment

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

If there are changes related to MySQL as well, can you move them in a separate PR?

Since, the PR title says it's for Postgres

@vs-dsva
Copy link

vs-dsva commented Aug 18, 2025

What is the status of this?
Can it be merged?

@JerryNixon
Copy link
Contributor

Resurrecting this with the PG team for review.

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.

6 participants