Repository navigation
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
Oracle stored-procedure comments are limited to service and dynamic_service modes while preserving existing SQL Server gating and comment deduplication. The completed static review returned no reportable findings.
🤖 Bits Code Review · Commit a33aa6a
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (9415) and master. ✅ No regressions detected |
BenchmarksBenchmark execution time: 2026-10-15 08:29:59 Comparing candidate commit a33aa6a in PR branch Found 0 performance improvements and 11 performance regressions! Performance is the same for 61 metrics, 0 unstable metrics, 75 known flaky benchmarks, 51 flaky benchmarks without significant changes.
|
tabgok
left a comment
There was a problem hiding this comment.
Approved from IDM's standpoint - please attain approval from at least one other language team as well
Summary of changes
Allow Oracle
CommandType.StoredProcedureinvocations to receive existing DBM service comments inservicemode and itsdynamic_servicealias.Reason for change
DBM/APM service mode was never fully implemented for the ODP.net integration. Multiple paying customers recently asked for this support.
Implementation details
Other details
Companion demo and results: https://github.com/DataDog/dbm/pull/620