Skip to content

Conversation

@florence-crl
Copy link
Contributor

@florence-crl florence-crl commented Dec 12, 2025

Fixes DOC-15477

  • In user-defined-function.md and stored-procedures.md, added Statement statistics section.
  • In routine-limitations.md, added limitation for statement diagnostics.
  • In statements-views.md, added callout that statements include those executed in UDF/SPs.
  • In transaction-diagnostics.md, added callout about limitation for statement diagnostics.

Rendered preview

… statistics section.

In routine-limitations.md, added limitation for statement diagnostics.

In statements-views.md, added callout that statements include those executed in UDF/SPs.

In transaction-diagnostics.md, added callout about limitation for statement diagnostics.
@github-actions
Copy link

@netlify
Copy link

netlify bot commented Dec 12, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit f52f77c
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/69442bb55339be0008396492

@netlify
Copy link

netlify bot commented Dec 12, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit f52f77c
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/69442bb52badaa0008960df4

@florence-crl
Copy link
Contributor Author

@kyle-a-wong , please review and let me know what is the issue number for the limitation. thanks!

@netlify
Copy link

netlify bot commented Dec 12, 2025

Netlify Preview

Name Link
🔨 Latest commit f52f77c
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/69442bb5a496b20008a731d5
😎 Deploy Preview https://deploy-preview-21621--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@kyle-a-wong kyle-a-wong left a comment

Choose a reason for hiding this comment

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

Nice, thanks for making the changes. LGTM

Copy link
Contributor

@taroface taroface left a comment

Choose a reason for hiding this comment

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

LGTM, nice additions! Some suggestions.

Copy link
Contributor Author

@florence-crl florence-crl left a comment

Choose a reason for hiding this comment

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

TFTRs!

@florence-crl florence-crl merged commit 75c0695 into main Dec 18, 2025
5 checks passed
@florence-crl florence-crl deleted the DOC-15477 branch December 18, 2025 18:27
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.

3 participants