Skip to content

Update aggregate-functions-transact-sql.md#2043

Merged
Jak-MS merged 1 commit intoMicrosoftDocs:livefrom
Nukgnaw:patch-1
May 17, 2019
Merged

Update aggregate-functions-transact-sql.md#2043
Jak-MS merged 1 commit intoMicrosoftDocs:livefrom
Nukgnaw:patch-1

Conversation

@Nukgnaw
Copy link
Copy Markdown
Contributor

@Nukgnaw Nukgnaw commented May 1, 2019

Add STRING_AGG to the list of functions that can not follow OVER clause.

Add STRING_AGG to the list of functions that can not follow OVER clause.
@PRMerger9
Copy link
Copy Markdown
Contributor

@Nukgnaw : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

@MashaMSFT
Copy link
Copy Markdown
Contributor

Hi @Nukgnaw , thank you for contacting Microsoft and providing your feedback to our documentation! We'll need some time to look into this further. Depending on the complexity, the turnaround time for this may vary anywhere from a few business days to a few business weeks. We'll reply via comment should we need anything further in the meantime, and please feel free to do the same.

Thanks again for your submission, and hope you have a wonderful rest of your day!

@Nukgnaw
Copy link
Copy Markdown
Contributor Author

Nukgnaw commented May 2, 2019

Hi @MashaMSFT, it might be helpful that I provide the error message I got when I tried to use STRING_AGG function followed by OVER clause: The function 'STRING_AGG' is not a valid windowing function, and cannot be used with the OVER clause.

Have a good day!

@MashaMSFT
Copy link
Copy Markdown
Contributor

Hi @Nukgnaw , thanks for your patience! I was able to confirm this is accurate, and appreciate your addition to the documentation. #sign-off

Thanks again and hope you have a wonderful weekend!

@Nukgnaw
Copy link
Copy Markdown
Contributor Author

Nukgnaw commented May 17, 2019

@MashaMSFT You are welcome! Have a wonderful weekend too!

@Jak-MS Jak-MS merged commit 68e128b into MicrosoftDocs:live May 17, 2019
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.

4 participants