Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update working-with-user-defined-types-in-sql-server.md#6703

Closed
alberto-morillo wants to merge 1 commit intoMicrosoftDocs:livefrom
alberto-morillo:patch-5
Closed

Update working-with-user-defined-types-in-sql-server.md#6703
alberto-morillo wants to merge 1 commit intoMicrosoftDocs:livefrom
alberto-morillo:patch-5

Conversation

@alberto-morillo
Copy link
Copy Markdown
Contributor

When we read the title of this page we expect to find examples of how to use user-defined data types with SQL Server.

When we read the title of this page we expect to find examples of how to use user-defined data types with SQL Server.
@PRMerger10
Copy link
Copy Markdown
Contributor

@alberto-morillo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Nov 19, 2021

@alberto-morillo Alberto, thanks for taking the time to create a new example! This is great. I need a little more time to test out your sample and get back to you on the change. Thanks for your patience!

@MikeRayMSFT MikeRayMSFT self-assigned this Feb 25, 2022
@MikeRayMSFT
Copy link
Copy Markdown
Contributor

@alberto-morillo - I reviewed this with @uc-msft

Would you mind modifying your example to not use cursor but instead execute a SELECT statement computing say MIN/MAX of the integer values.

Thank you,
Mike

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented May 6, 2022

@alberto-morillo I think Mike Ray has a good point about the sample being stronger if we avoided cursors. Having not heard back from you on this, I am going to close the public PR for now, but if you want to look at the suggestion and propose a modified sample, I can help test and publish it. We can reopen this PR and continue working at any time. Thanks!

@rothja rothja closed this May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants