Improve Git commit info building #9210
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
daniel-mohedano added type: enhancement 
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 11 unstable metrics. Startup time reports for insecure-bankgantt title insecure-bank - global startup overhead: candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section tracing Agent [baseline] (1.042 s) : 0, 1042391 Total [baseline] (8.587 s) : 0, 8586903 Agent [candidate] (1.042 s) : 0, 1042423 Total [candidate] (8.638 s) : 0, 8637580 section iast Agent [baseline] (1.173 s) : 0, 1173210 Total [baseline] (9.316 s) : 0, 9316374 Agent [candidate] (1.173 s) : 0, 1173011 Total [candidate] (9.264 s) : 0, 9263568
gantt title insecure-bank - break down per module: candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section tracing crashtracking [baseline] (1.431 ms) : 0, 1431 crashtracking [candidate] (1.428 ms) : 0, 1428 BytebuddyAgent [baseline] (729.575 ms) : 0, 729575 BytebuddyAgent [candidate] (732.685 ms) : 0, 732685 GlobalTracer [baseline] (241.256 ms) : 0, 241256 GlobalTracer [candidate] (241.996 ms) : 0, 241996 AppSec [baseline] (30.428 ms) : 0, 30428 AppSec [candidate] (30.424 ms) : 0, 30424 Debugger [baseline] (6.011 ms) : 0, 6011 Debugger [candidate] (6.0 ms) : 0, 6000 Remote Config [baseline] (646.697 µs) : 0, 647 Remote Config [candidate] (645.221 µs) : 0, 645 Telemetry [baseline] (12.079 ms) : 0, 12079 Telemetry [candidate] (8.272 ms) : 0, 8272 section iast crashtracking [baseline] (1.43 ms) : 0, 1430 crashtracking [candidate] (1.422 ms) : 0, 1422 BytebuddyAgent [baseline] (847.545 ms) : 0, 847545 BytebuddyAgent [candidate] (846.998 ms) : 0, 846998 GlobalTracer [baseline] (231.132 ms) : 0, 231132 GlobalTracer [candidate] (231.365 ms) : 0, 231365 AppSec [baseline] (27.208 ms) : 0, 27208 AppSec [candidate] (28.175 ms) : 0, 28175 Debugger [baseline] (5.726 ms) : 0, 5726 Debugger [candidate] (6.618 ms) : 0, 6618 Remote Config [baseline] (593.995 µs) : 0, 594 Remote Config [candidate] (588.988 µs) : 0, 589 Telemetry [baseline] (8.047 ms) : 0, 8047 Telemetry [candidate] (7.842 ms) : 0, 7842 IAST [baseline] (30.651 ms) : 0, 30651 IAST [candidate] (29.127 ms) : 0, 29127 Startup time reports for petclinicgantt title petclinic - global startup overhead: candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section tracing Agent [baseline] (1.046 s) : 0, 1046495 Total [baseline] (10.754 s) : 0, 10753710 Agent [candidate] (1.039 s) : 0, 1039007 Total [candidate] (10.661 s) : 0, 10660685 section appsec Agent [baseline] (1.219 s) : 0, 1219257 Total [baseline] (10.761 s) : 0, 10761131 Agent [candidate] (1.22 s) : 0, 1220238 Total [candidate] (10.868 s) : 0, 10867957 section iast Agent [baseline] (1.187 s) : 0, 1187429 Total [baseline] (10.965 s) : 0, 10964526 Agent [candidate] (1.182 s) : 0, 1181507 Total [candidate] (10.973 s) : 0, 10973239 section profiling Agent [baseline] (1.191 s) : 0, 1191097 Total [baseline] (10.905 s) : 0, 10905011 Agent [candidate] (1.182 s) : 0, 1182408 Total [candidate] (10.797 s) : 0, 10796740
gantt title petclinic - break down per module: candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section tracing crashtracking [baseline] (1.435 ms) : 0, 1435 crashtracking [candidate] (1.432 ms) : 0, 1432 BytebuddyAgent [baseline] (731.492 ms) : 0, 731492 BytebuddyAgent [candidate] (729.695 ms) : 0, 729695 GlobalTracer [baseline] (241.977 ms) : 0, 241977 GlobalTracer [candidate] (241.622 ms) : 0, 241622 AppSec [baseline] (30.519 ms) : 0, 30519 AppSec [candidate] (30.336 ms) : 0, 30336 Debugger [baseline] (5.995 ms) : 0, 5995 Debugger [candidate] (6.039 ms) : 0, 6039 Remote Config [baseline] (654.917 µs) : 0, 655 Remote Config [candidate] (651.003 µs) : 0, 651 Telemetry [baseline] (13.34 ms) : 0, 13340 Telemetry [candidate] (8.206 ms) : 0, 8206 section appsec crashtracking [baseline] (1.44 ms) : 0, 1440 crashtracking [candidate] (1.428 ms) : 0, 1428 BytebuddyAgent [baseline] (751.72 ms) : 0, 751720 BytebuddyAgent [candidate] (753.516 ms) : 0, 753516 GlobalTracer [baseline] (235.404 ms) : 0, 235404 GlobalTracer [candidate] (234.724 ms) : 0, 234724 AppSec [baseline] (168.296 ms) : 0, 168296 AppSec [candidate] (169.137 ms) : 0, 169137 Debugger [baseline] (8.547 ms) : 0, 8547 Debugger [candidate] (7.893 ms) : 0, 7893 Remote Config [baseline] (613.813 µs) : 0, 614 Remote Config [candidate] (602.078 µs) : 0, 602 Telemetry [baseline] (8.223 ms) : 0, 8223 Telemetry [candidate] (8.095 ms) : 0, 8095 IAST [baseline] (23.687 ms) : 0, 23687 IAST [candidate] (23.585 ms) : 0, 23585 section iast crashtracking [baseline] (1.446 ms) : 0, 1446 crashtracking [candidate] (1.447 ms) : 0, 1447 BytebuddyAgent [baseline] (858.616 ms) : 0, 858616 BytebuddyAgent [candidate] (853.973 ms) : 0, 853973 GlobalTracer [baseline] (232.807 ms) : 0, 232807 GlobalTracer [candidate] (232.361 ms) : 0, 232361 AppSec [baseline] (26.018 ms) : 0, 26018 AppSec [candidate] (28.378 ms) : 0, 28378 Debugger [baseline] (7.56 ms) : 0, 7560 Debugger [candidate] (5.778 ms) : 0, 5778 Remote Config [baseline] (594.688 µs) : 0, 595 Remote Config [candidate] (592.185 µs) : 0, 592 Telemetry [baseline] (8.208 ms) : 0, 8208 Telemetry [candidate] (7.978 ms) : 0, 7978 IAST [baseline] (31.038 ms) : 0, 31038 IAST [candidate] (29.945 ms) : 0, 29945 section profiling crashtracking [baseline] (1.398 ms) : 0, 1398 crashtracking [candidate] (1.393 ms) : 0, 1393 BytebuddyAgent [baseline] (759.479 ms) : 0, 759479 BytebuddyAgent [candidate] (757.42 ms) : 0, 757420 GlobalTracer [baseline] (220.866 ms) : 0, 220866 GlobalTracer [candidate] (220.11 ms) : 0, 220110 AppSec [baseline] (30.194 ms) : 0, 30194 AppSec [candidate] (30.153 ms) : 0, 30153 Debugger [baseline] (6.165 ms) : 0, 6165 Debugger [candidate] (6.297 ms) : 0, 6297 Remote Config [baseline] (688.63 µs) : 0, 689 Remote Config [candidate] (667.053 µs) : 0, 667 Telemetry [baseline] (15.172 ms) : 0, 15172 Telemetry [candidate] (11.838 ms) : 0, 11838 ProfilingAgent [baseline] (108.767 ms) : 0, 108767 ProfilingAgent [candidate] (106.184 ms) : 0, 106184 Profiling [baseline] (108.791 ms) : 0, 108791 Profiling [candidate] (106.207 ms) : 0, 106207 LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 2 performance regressions! Performance is the same for 10 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt title petclinic - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section baseline no_agent (37.027 ms) : 36737, 37317 . : milestone, 37027, appsec (47.2 ms) : 46774, 47626 . : milestone, 47200, code_origins (45.184 ms) : 44792, 45576 . : milestone, 45184, iast (45.262 ms) : 44858, 45666 . : milestone, 45262, profiling (46.354 ms) : 45921, 46786 . : milestone, 46354, tracing (44.436 ms) : 44062, 44810 . : milestone, 44436, section candidate no_agent (36.539 ms) : 36249, 36828 . : milestone, 36539, appsec (49.866 ms) : 49418, 50314 . : milestone, 49866, code_origins (45.931 ms) : 45541, 46320 . : milestone, 45931, iast (45.285 ms) : 44894, 45676 . : milestone, 45285, profiling (48.437 ms) : 47927, 48948 . : milestone, 48437, tracing (44.231 ms) : 43841, 44620 . : milestone, 44231,
Request duration reports for insecure-bankgantt title insecure-bank - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section baseline no_agent (4.392 ms) : 4342, 4442 . : milestone, 4392, iast (9.417 ms) : 9255, 9579 . : milestone, 9417, iast_FULL (14.085 ms) : 13807, 14364 . : milestone, 14085, iast_GLOBAL (10.158 ms) : 9982, 10335 . : milestone, 10158, profiling (8.907 ms) : 8758, 9057 . : milestone, 8907, tracing (7.351 ms) : 7241, 7460 . : milestone, 7351, section candidate no_agent (4.432 ms) : 4380, 4485 . : milestone, 4432, iast (9.256 ms) : 9103, 9409 . : milestone, 9256, iast_FULL (13.766 ms) : 13495, 14038 . : milestone, 13766, iast_GLOBAL (9.994 ms) : 9823, 10166 . : milestone, 9994, profiling (8.608 ms) : 8465, 8752 . : milestone, 8608, tracing (7.456 ms) : 7346, 7566 . : milestone, 7456,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for biojavagantt title biojava - execution time [CI 0.99] : candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section baseline no_agent (15.118 s) : 15118000, 15118000 . : milestone, 15118000, appsec (14.76 s) : 14760000, 14760000 . : milestone, 14760000, iast (18.779 s) : 18779000, 18779000 . : milestone, 18779000, iast_GLOBAL (18.364 s) : 18364000, 18364000 . : milestone, 18364000, profiling (16.04 s) : 16040000, 16040000 . : milestone, 16040000, tracing (14.691 s) : 14691000, 14691000 . : milestone, 14691000, section candidate no_agent (15.188 s) : 15188000, 15188000 . : milestone, 15188000, appsec (15.092 s) : 15092000, 15092000 . : milestone, 15092000, iast (18.573 s) : 18573000, 18573000 . : milestone, 18573000, iast_GLOBAL (18.074 s) : 18074000, 18074000 . : milestone, 18074000, profiling (15.719 s) : 15719000, 15719000 . : milestone, 15719000, tracing (14.966 s) : 14966000, 14966000 . : milestone, 14966000,
Execution time for tomcatgantt title tomcat - execution time [CI 0.99] : candidate=1.52.0-SNAPSHOT~004fc73ce9, baseline=1.51.1-SNAPSHOT~6fc7077594 dateFormat X axisFormat %s section baseline no_agent (1.469 ms) : 1458, 1481 . : milestone, 1469, appsec (3.654 ms) : 3438, 3870 . : milestone, 3654, iast (2.19 ms) : 2128, 2252 . : milestone, 2190, iast_GLOBAL (2.234 ms) : 2171, 2296 . : milestone, 2234, profiling (2.041 ms) : 1990, 2092 . : milestone, 2041, tracing (2.001 ms) : 1952, 2049 . : milestone, 2001, section candidate no_agent (1.477 ms) : 1466, 1489 . : milestone, 1477, appsec (2.455 ms) : 2402, 2508 . : milestone, 2455, iast (2.183 ms) : 2121, 2245 . : milestone, 2183, iast_GLOBAL (2.227 ms) : 2164, 2289 . : milestone, 2227, profiling (2.027 ms) : 1977, 2076 . : milestone, 2027, tracing (1.998 ms) : 1950, 2046 . : milestone, 1998,
|
tonyredondo approved these changes Jul 22, 2025
| Object number = pullRequest.get("number"); | ||
| if (number != null) { | ||
| prNumber = | ||
| number instanceof Double |
There was a problem hiding this comment.
Can it really be a Double?
There was a problem hiding this comment.
Not really, but moshi defaults to treating Objects as Double when they are numeric. I now realize that the if check doesn't make much sense given that it is always treated as a Double. I will simpllify it 👍
| .executeCommand( | ||
| IOUtils::readFully, | ||
| "git", | ||
| "show", |
There was a problem hiding this comment.
Just to double-check - this doesn't require a newer git CLI version than the previous command, does it?
There was a problem hiding this comment.
It uses the same placeholders as before (with the addition of %H for the full commit hash) but bundled together in the same command, and there is no apparent difference between show and log regarding git versions, so we should be in the clear
| } | ||
| | ||
| public synchronized boolean unshallow(boolean parentOnly) | ||
| public synchronized boolean unshallow(@Nullable String commit) |
There was a problem hiding this comment.
commit is the oldest commit that we want to fetch, right? Could you please add a brief Javadoc explaining this, and the behaviour when the passed commit is null?
| public synchronized boolean unshallow(boolean parentOnly) | ||
| public synchronized boolean unshallow(@Nullable String commit) | ||
| throws IOException, InterruptedException, TimeoutException { | ||
| if (!config.isCiVisibilityGitUnshallowEnabled() || !gitClient.isShallow()) { |
There was a problem hiding this comment.
Not sure if the gitClient.isShallow() needs to be updated to check if the provided commit SHA is present. Also, what happens if we request unshallow with a commit SHA, and then request "full unshallow" with a null SHA?
There was a problem hiding this comment.
Will look into updating the logic to check isShallow for a specific commit if the argument is provided. Could avoid additional git cmd calls that are unnecessary if it is called repeatedly. Regarding the two calls, AFAIK the second "full unshallow" would just be executed as regular, unshallowing to the shallow-since limit, given that the isShallow check would be true.
nikita-tkachenko-datadog approved these changes Jul 23, 2025
| return; | ||
| } | ||
| executeCommand( | ||
| Command.OTHER, |
There was a problem hiding this comment.
Perhaps it makes sense to use a new tag value here - e.g. Command.FETCH_COMMIT - to be able to distinguish these commands from the others in telemetry, as they can potentially be slow and we want to be able to see that
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Aug 5, 2025
| Package | Type | Package file | Manager | Update | Change | |---|---|---|---|---|---| | [redis.clients:jedis](https://github.com/redis/jedis) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `6.0.0` -> `6.1.0` | | [com.google.api.grpc:proto-google-common-protos](https://github.com/googleapis/sdk-platform-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.59.2` -> `2.60.0` | | [com.google.cloud:google-cloud-core-http](https://github.com/googleapis/sdk-platform-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.58.2` -> `2.59.0` | | [com.google.cloud:google-cloud-core](https://github.com/googleapis/sdk-platform-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.58.2` -> `2.59.0` | | [com.google.api:gax](https://github.com/googleapis/sdk-platform-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `2.68.2` -> `2.69.0` | | [com.squareup.wire](https://github.com/square/wire) | plugin | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-schema](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-runtime](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-reflector](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-moshi-adapter](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-grpc-client](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.squareup.wire:wire-bom](https://github.com/square/wire) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `5.3.5` -> `5.3.6` | | [com.datadoghq:dd-trace-api](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.51.2` -> `1.52.0` | | [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:sqs](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:regions](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | | [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) | dependencies | misk/gradle/libs.versions.toml | gradle | patch | `2.32.14` -> `2.32.15` | --- ### Release Notes <details> <summary>redis/jedis (redis.clients:jedis)</summary> ### [`v6.1.0`](https://github.com/redis/jedis/releases/tag/v6.1.0): 6.1.0 ### Changes #### 🚀 New Features - Add support for SVS-VAMANA vector indexing ([#​4222](redis/jedis#4222)) - Clarify why new stream entries aren't deleted with XDELEX ([#​4218](redis/jedis#4218)) - Add support for new stream commands ([#​4211](redis/jedis#4211)) - Add Support for New BITOP Operations in Redis 8.2 ([#​4188](redis/jedis#4188)) ([#​4190](redis/jedis#4190)) - Add binary stream support for XREAD and XREADGROUP ([#​3566](redis/jedis#3566)) ([#​4152](redis/jedis#4152)) - Run pipeline in current thread if all the keys on same node ([#​4149](redis/jedis#4149)) #### 🐛 Bug Fixes - Restore binary compatibility of SetParams ([#​4225](redis/jedis#4225)) - Fix memory leak in JedisClusterInfoCache - replica nodes not cleared ([#​4205](redis/jedis#4205)) - Fix:JedisCluster throws NullPointerException when maxAttempts is set to 0 ([#​4186](redis/jedis#4186)) #### 🧰 Maintenance - DOC-5471 time series doc examples ([#​4210](redis/jedis#4210)) - Bump jackson.version from 2.19.1 to 2.19.2 ([#​4208](redis/jedis#4208)) - Fix flaky test ClientCommandsTest.killSkipmeYesNo ([#​4206](redis/jedis#4206)) - Bump org.junit:junit-bom from 5.13.2 to 5.13.3 ([#​4198](redis/jedis#4198)) - Migrate publishing to Maven Central Portal ([#​4199](redis/jedis#4199)) - Bump org.apache.maven.plugins:maven-gpg-plugin from 3.2.7 to 3.2.8 ([#​4197](redis/jedis#4197)) - Bump org.junit:junit-bom from 5.13.1 to 5.13.2 ([#​4192](redis/jedis#4192)) - DOC-5227 added probabilistic data type examples ([#​4184](redis/jedis#4184)) - Bump jackson.version from 2.19.0 to 2.19.1 ([#​4180](redis/jedis#4180)) - Update test infra to use latest Redis ([#​4179](redis/jedis#4179)) - Bump org.junit:junit-bom from 5.13.0-RC1 to 5.13.1 ([#​4174](redis/jedis#4174)) - Bump org.json:json from [`2025010`](redis/jedis@20250107) to [`2025051`](redis/jedis@20250517) ([#​4171](redis/jedis#4171)) - Bump org.apache.httpcomponents.client5:httpclient5-fluent from 5.4.4 to 5.5 ([#​4170](redis/jedis#4170)) - Fix flaky tests in DocumentTest ([#​3617](redis/jedis#3617)) - Add retryable command execution example ([#​3780](redis/jedis#3780)) - Bump jackson.version from 2.18.3 to 2.19.0 ([#​4160](redis/jedis#4160)) - Bump com.google.code.gson:gson from 2.12.1 to 2.13.1 ([#​4161](redis/jedis#4161)) #### Contributors We'd like to thank all the contributors who worked on this release! [@​219sansim](https://github.com/219sansim), [@​YoHanKi](https://github.com/YoHanKi), [@​andy-stark-redis](https://github.com/andy-stark-redis), [@​ggivo](https://github.com/ggivo), [@​jujn](https://github.com/jujn), [@​thachlp](https://github.com/thachlp), [@​uglide](https://github.com/uglide) and [@​xrayw](https://github.com/xrayw) </details> <details> <summary>googleapis/sdk-platform-java (com.google.api.grpc:proto-google-common-protos)</summary> ### [`v2.60.0`](https://github.com/googleapis/sdk-platform-java/blob/HEAD/CHANGELOG.md#2600-2025-06-23) ##### Features - handle auto pagination for BigQuery v2 ([#​3829](googleapis/sdk-platform-java#3829)) ([025c84c](googleapis/sdk-platform-java@025c84c)) ##### Dependencies - update google auth library dependencies to v1.37.1 ([#​3846](googleapis/sdk-platform-java#3846)) ([ea1d9e5](googleapis/sdk-platform-java@ea1d9e5)) - update google http client dependencies to v1.47.1 ([#​3848](googleapis/sdk-platform-java#3848)) ([a9a39d7](googleapis/sdk-platform-java@a9a39d7)) </details> <details> <summary>square/wire (com.squareup.wire)</summary> ### [`v5.3.6`](https://github.com/square/wire/blob/HEAD/CHANGELOG.md#Version-536) [Compare Source](square/wire@5.3.5...5.3.6) *2025-08-05* ##### CLI - New CLI option `--ignore_unused_roots_and_prunes` ([#​3354](square/wire#3354)) ##### JVM - Fix: Handle negative hexadecimal in default values ([#​3355](square/wire#3355)) - Optimization: Avoid copying of repeated and map types when mutableTypes are being used ([#​3352](square/wire#3352) by \[Rahul Ravikumar]\[tikurahul]) ##### Swift - Fix: Properly disambiguate OneOf enum if it has the same name as enclosing type ([#​3350](square/wire#3350) by \[Dimitris Koutsogiorgas]\[dnkoutso]) </details> <details> <summary>datadog/dd-trace-java (com.datadoghq:dd-trace-api)</summary> ### [`v1.52.0`](https://github.com/DataDog/dd-trace-java/releases/tag/v1.52.0): 1.52.0 ### Components #### Application Security Management (WAF) - ✨ Only report ASM\_DD, ASM\_DATA and ASM capabilities when AppSec is enabled ([#​9260](DataDog/dd-trace-java#9260) - [@​manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - 🐛 Fix NPE in AppSecConfigServiceImpl ([#​9165](DataDog/dd-trace-java#9165) - [@​manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) - 🐛 Fix AppSec play.mvc.StatusHeader instrumentation for play 2.6 ([#​9160](DataDog/dd-trace-java#9160) - [@​manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) #### Build & Tooling - ✨📖 Automatically register crashtracking via native extensions ([#​8851](DataDog/dd-trace-java#8851) - [@​MattAlp](https://github.com/MattAlp)) #### Configuration at Runtime - ✨ Create activation origin config for telemetry ([#​9064](DataDog/dd-trace-java#9064) - [@​sezen-datadog](https://github.com/sezen-datadog)) #### Continuous Integration Visibility - ✨ Update GitLab provided tags ([#​9275](DataDog/dd-trace-java#9275) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) - 🐛 Fix base branch SHA usage in GitHub Actions ([#​9257](DataDog/dd-trace-java#9257) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Add `ci.job.id` tag ([#​9256](DataDog/dd-trace-java#9256) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Add new org to Weaver instrumentation ([#​9235](DataDog/dd-trace-java#9235) - [@​daniel-mohedano](https://github.com/daniel-mohedano) - thanks for the contribution!) - ✨ Improve Git commit info building ([#​9210](DataDog/dd-trace-java#9210) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Update Attempt to Fix to v5 ([#​9145](DataDog/dd-trace-java#9145) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) #### Crash tracking - ✨📖 Automatically register crashtracking via native extensions ([#​8851](DataDog/dd-trace-java#8851) - [@​MattAlp](https://github.com/MattAlp)) #### Data Streams Monitoring - ✨ Reduce DSM CPU overheard ([#​9151](DataDog/dd-trace-java#9151) - [@​kr-igor](https://github.com/kr-igor)) - ✨⚡ DSM optimizations for high throughput scenarios ([#​9137](DataDog/dd-trace-java#9137) - [@​kr-igor](https://github.com/kr-igor)) #### Database Monitoring - 🐛 Fix duplicate trace injection for SQL Server and Oracle DBM full propagation mode ([#​9224](DataDog/dd-trace-java#9224) - [@​lu-zhengda](https://github.com/lu-zhengda)) #### Dynamic Instrumentation - 🐛 Add URI in string primitives ([#​9285](DataDog/dd-trace-java#9285) - [@​jpbempel](https://github.com/jpbempel)) - ✨ Increase SourceFile tracking max queue size ([#​9271](DataDog/dd-trace-java#9271) - [@​jpbempel](https://github.com/jpbempel)) - ✨ Add capping on SourceFile tracking queue ([#​9245](DataDog/dd-trace-java#9245) - [@​jpbempel](https://github.com/jpbempel)) - ✨ Add third-party filtering in SourceFile tracking ([#​9205](DataDog/dd-trace-java#9205) - [@​jpbempel](https://github.com/jpbempel)) #### ML Observability (LLMObs) - ✨ Add methods to capture embedding and retrieval spans ([#​9297](DataDog/dd-trace-java#9297) - [@​nayeem-kamal](https://github.com/nayeem-kamal)) #### Metrics - ✨ Change primary client stats configuration key ([#​9196](DataDog/dd-trace-java#9196) - [@​amarziali](https://github.com/amarziali)) - ✨ Calculate client stats also if the span kind is eligible ([#​9157](DataDog/dd-trace-java#9157) - [@​amarziali](https://github.com/amarziali)) - ✨ Backpropagate peer tags ([#​9144](DataDog/dd-trace-java#9144) - [@​bric3](https://github.com/bric3)) - 🐛 Make client stats reliable in case of downgrade ([#​9136](DataDog/dd-trace-java#9136) - [@​amarziali](https://github.com/amarziali)) #### Platform components - 🐛 Fix VM options parsing from /proc/fs ([#​9255](DataDog/dd-trace-java#9255) - [@​PerfectSlayer](https://github.com/PerfectSlayer)) #### Profiling - ✨ Switch profile compression to zstd default ([#​9293](DataDog/dd-trace-java#9293) - [@​jbachorik](https://github.com/jbachorik)) - ✨ Bump ddprof to 1.29.0 ([#​9262](DataDog/dd-trace-java#9262) - [@​zhengyu123](https://github.com/zhengyu123)) - Potential memory leak and race with the JVMTI wallclock sampler by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#234 - Downport async-profiler no-allocation changes by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#245 - Adopt openjdk safefetch by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#246 - Safe fetch 64-bit value and pointer by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#247 - Rebase on Async-Profiler 4.1 by [@​jbachorik](https://github.com/jbachorik) in DataDog/java-profiler#252 - Patch upstream stackWalker.cpp not to fail on unaligned access by [@​jbachorik](https://github.com/jbachorik) in DataDog/java-profiler#218 - Remap thread id to avoid bitmap contention by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#229 - Improve performance using Unsafe to activate/deactivate thread filter by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#230 - Unify context propagation by [@​zhengyu123](https://github.com/zhengyu123) in DataDog/java-profiler#231 - 🐛 Fix the profiler stackdepth setting propagation in recent (22+) Java versions ([#​9130](DataDog/dd-trace-java#9130) - [@​jbachorik](https://github.com/jbachorik)) #### Realtime User Monitoring - ✨ Wrap servlet original PrintWriter on rum injector ([#​9146](DataDog/dd-trace-java#9146) - [@​amarziali](https://github.com/amarziali)) #### Telemetry - ✨ Create activation origin config for telemetry ([#​9064](DataDog/dd-trace-java#9064) - [@​sezen-datadog](https://github.com/sezen-datadog)) #### Tracer core - ✨⚡ Reduce span construction overhead by switching to optimized TagMap ([#​8589](DataDog/dd-trace-java#8589) - [@​dougqh](https://github.com/dougqh)) - 🐛 Match Hands Off Config selectors on process\_arguments value ([#​9201](DataDog/dd-trace-java#9201) - [@​paullegranddc](https://github.com/paullegranddc)) - ✨ Move JSON generation to sender thread to improve startup time. ([#​9197](DataDog/dd-trace-java#9197) - [@​AlexeyKuznetsov-DD](https://github.com/AlexeyKuznetsov-DD)) - ✨ Improve agent to avoid loading global config on main thread ([#​9190](DataDog/dd-trace-java#9190) - [@​PerfectSlayer](https://github.com/PerfectSlayer)) - ✨ add injection metadata fields to telemetry forwarder ([#​9185](DataDog/dd-trace-java#9185) - [@​sydney-tung](https://github.com/sydney-tung)) - 🐛 Avoid race conditions on feature discovery during Writer creation ([#​9173](DataDog/dd-trace-java#9173) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) - ✨ Surface potential root cause when agent initialization errors ([#​9170](DataDog/dd-trace-java#9170) - [@​AlexeyKuznetsov-DD](https://github.com/AlexeyKuznetsov-DD)) - 💡 Support adding W3C baggage as span tags ([#​9169](DataDog/dd-trace-java#9169) - [@​rachelyangdog](https://github.com/rachelyangdog)) - ✨⚡ Align our default classloader excludes with OTel ([#​9161](DataDog/dd-trace-java#9161) - [@​mcculls](https://github.com/mcculls)) - ✨ Backpropagate container tags hash coming from the info endpoint ([#​9156](DataDog/dd-trace-java#9156) - [@​amarziali](https://github.com/amarziali)) - 🐛 Avoid race conditions and multiple agent discovery feature states ([#​9135](DataDog/dd-trace-java#9135) - [@​amarziali](https://github.com/amarziali)) ### Instrumentations #### AWS SDK instrumentation - ✨ Enhance Service Representation for Serverless ([#​9203](DataDog/dd-trace-java#9203) - [@​zarirhamza](https://github.com/zarirhamza)) #### gRPC instrumentation - 🐛 Add check to prevent injection of repeated GRPC headers ([#​9246](DataDog/dd-trace-java#9246) - [@​mhlidd](https://github.com/mhlidd)) #### JDBC instrumentation - 🐛 Fix duplicate trace injection for SQL Server and Oracle DBM full propagation mode ([#​9224](DataDog/dd-trace-java#9224) - [@​lu-zhengda](https://github.com/lu-zhengda)) #### Play Framework instrumentation - 🐛 Fix AppSec play.mvc.StatusHeader instrumentation for play 2.6 ([#​9160](DataDog/dd-trace-java#9160) - [@​manuel-alvarez-alvarez](https://github.com/manuel-alvarez-alvarez)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am every weekday" in timezone Australia/Melbourne, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Never, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). GitOrigin-RevId: 9da36329ffa552291a640381780b608ef6513e29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
What Does This Do
CommitInfoby the ShellGitClient to use a single git command, instead of a separate command for each field.--no-write-fetch-headinstead of--deepen=1.pr.numberin Github Actions json file.Additional Notes
Tested on Github PR:
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: SDTEST-2261