Skip to content

[Internal] Query: Adds missing function names SqlIdentifier for SQL DOM#5262

Merged
microsoft-github-policy-service[bot] merged 4 commits intomasterfrom
users/leminh/AddMissingSystemFunctionIdentifier
Jul 11, 2025
Merged

[Internal] Query: Adds missing function names SqlIdentifier for SQL DOM#5262
microsoft-github-policy-service[bot] merged 4 commits intomasterfrom
users/leminh/AddMissingSystemFunctionIdentifier

Conversation

@leminh98
Copy link
Copy Markdown
Contributor

Pull Request Template

Description

This PR adds missing constants for function names in SqlIdentifier class.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • [] New feature (non-breaking change which adds functionality)
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

Closing issues

To automatically close an issue: closes #IssueNumber

@adityasa
Copy link
Copy Markdown
Contributor

//------------------------------------------------------------

Can you add coverage?


Refers to: Microsoft.Azure.Cosmos/src/SqlObjects/SqlFunctionCallScalarExpression.cs:1 in 935737a. [](commit_id = 935737a, deletion_comment = False)

Copy link
Copy Markdown
Contributor

@adityasa adityasa left a comment

Choose a reason for hiding this comment

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

:shipit:

@leminh98 leminh98 added the auto-merge Enables automation to merge PRs label Jul 11, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot enabled auto-merge (squash) July 11, 2025 18:12
@microsoft-github-policy-service microsoft-github-policy-service Bot merged commit 4c82ee6 into master Jul 11, 2025
27 checks passed
@microsoft-github-policy-service microsoft-github-policy-service Bot deleted the users/leminh/AddMissingSystemFunctionIdentifier branch July 11, 2025 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Enables automation to merge PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants